From b5fbb94cfc4c34a9176b5e49cd33eb229ac5bd31 Mon Sep 17 00:00:00 2001
From: Alireza Ahmadi <alireza7@gmail.com>
Date: Sat, 10 Oct 2026 23:00:52 +0200
Subject: [PATCH] Subscriptions: offer MASQUE clients only the JSON
 subscription

---
 web/assets/js/model/dbinbound.js     |  4 ++++
 web/html/common/qrcode_modal.html    |  3 ++-
 web/html/xui/inbound_info_modal.html |  3 ++-
 web/html/xui/inbounds.html           | 10 ++++++++--
 4 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/web/assets/js/model/dbinbound.js b/web/assets/js/model/dbinbound.js
index 57bc7c08..e39181a0 100644
--- a/web/assets/js/model/dbinbound.js
+++ b/web/assets/js/model/dbinbound.js
@@ -60,6 +60,10 @@ class DBInbound {
         return this.protocol === Protocols.WIREGUARD;
     }
 
+    get isMasque() {
+        return this.protocol === Protocols.MASQUE;
+    }
+
     get address() {
         let address = location.hostname;
         if (!ObjectUtil.isEmpty(this.listen) && this.listen !== "0.0.0.0") {
diff --git a/web/html/common/qrcode_modal.html b/web/html/common/qrcode_modal.html
index 19479bd9..fb111bf3 100644
--- a/web/html/common/qrcode_modal.html
+++ b/web/html/common/qrcode_modal.html
@@ -34,7 +34,8 @@
          width="560px" :class="themeSwitcher.currentTheme">
     <a-collapse v-model="qrModal.activeKey">
         <template v-if="app.subSettings.enable && qrModal.subId">
-            <a-collapse-panel key="sub" :force-render="true">
+            <!-- MASQUE has no share link format, so only the JSON subscription carries it -->
+            <a-collapse-panel v-if="qrModal.dbInbound.protocol !== Protocols.MASQUE" key="sub" :force-render="true">
                 <span slot="header">{{ i18n "pages.settings.subSettings" }}</span>
                 <div class="qr-panel-actions">
                     <a-button size="small" icon="copy" @click.stop="copyToClipboard(genSubLink(qrModal.subId))"></a-button>
diff --git a/web/html/xui/inbound_info_modal.html b/web/html/xui/inbound_info_modal.html
index 23dbed97..6ba480e9 100644
--- a/web/html/xui/inbound_info_modal.html
+++ b/web/html/xui/inbound_info_modal.html
@@ -206,7 +206,8 @@
         </table>
         <template v-if="app.subSettings.enable && infoModal.clientSettings.subId && !dbInbound.isWireguard">
             <a-divider>Subscription URL</a-divider>
-            <a-row>
+            <!-- MASQUE has no share link format, so only the JSON subscription carries it -->
+            <a-row v-if="!dbInbound.isMasque">
                 <a-col :sx="24" :md="22">SUB: <a :href="[[ infoModal.subLink ]]" target="_blank">[[ infoModal.subLink ]]</a></a-col>
                 <a-col :sx="24" :md="2" style="text-align: right;">
                     <a-tooltip title='{{ i18n "copy" }}'>
diff --git a/web/html/xui/inbounds.html b/web/html/xui/inbounds.html
index 1c006711..7935178d 100644
--- a/web/html/xui/inbounds.html
+++ b/web/html/xui/inbounds.html
@@ -1291,6 +1291,12 @@
                 newDbInbound = this.checkFallback(dbInbound);
                 txtModal.show('{{ i18n "pages.inbounds.export"}}', newDbInbound.genInboundLinks(this.remarkModel), newDbInbound.remark);
             },
+            // MASQUE has no share link format, so its clients only work through
+            // the JSON subscription; the plain one answers them with an error.
+            subLinkFor(dbInbound, subId) {
+                if (dbInbound.protocol === Protocols.MASQUE) return this.subSettings.subJsonURI + subId;
+                return this.subSettings.subURI + subId + "?name=" + subId;
+            },
             exportSubs(dbInboundId) {
                 const dbInbound = this.dbInbounds.find(row => row.id === dbInboundId);
                 const clients = this.getInboundClients(dbInbound);
@@ -1298,7 +1304,7 @@
                 if (clients != null){
                     clients.forEach(c => {
                         if (c.subId && c.subId.length>0){
-                            subLinks.push(this.subSettings.subURI + c.subId + "?name=" + c.subId)
+                            subLinks.push(this.subLinkFor(dbInbound, c.subId))
                         }
                     })
                 }
@@ -1325,7 +1331,7 @@
                     if (clients != null){
                         clients.forEach(c => {
                             if (c.subId && c.subId.length>0){
-                                subLinks.push(this.subSettings.subURI + c.subId + "?name=" + c.subId)
+                                subLinks.push(this.subLinkFor(dbInbound, c.subId))
                             }
                         })
                     }