diff --git a/src/js/yunohost/controllers/firewall.js b/src/js/yunohost/controllers/firewall.js
index 5dfbeeff..dfb2300d 100644
--- a/src/js/yunohost/controllers/firewall.js
+++ b/src/js/yunohost/controllers/firewall.js
@@ -38,7 +38,7 @@
     app.get('#/tools/firewall/upnp/:action', function (c) {
         c.confirm(
             y18n.t('firewall'),
-            y18n.t('confirm_upnp_action', [y18n.t(c.params['action'])]),
+            y18n.t('confirm_upnp_' + c.params['action'].toLowerCase()),
             function(){
                 params = {'action' : c.params['action']};
                 c.api('/firewall/upnp', function(data) {
diff --git a/src/locales/en.json b/src/locales/en.json
index c01aec86..4e5e4998 100644
--- a/src/locales/en.json
+++ b/src/locales/en.json
@@ -79,6 +79,8 @@
     "confirm_update_apps": "Are you sure you want update every applications ?",
     "confirm_update_packages": "Are you sure you want update every packages ?",
     "confirm_upnp_action": "Are you sure you want to %s UPnP ?",
+    "confirm_upnp_enable": "Are you sure you want to enable UPnP ?",
+    "confirm_upnp_disable": "Are you sure you want to disable UPnP ?",
     "connection": "Connection",
     "copy": "Copy",
     "count_min": "%s min",