From bc1e6315a86de7263f1e70be8f19db712fe4d310 Mon Sep 17 00:00:00 2001
From: Chang lue Tsen <tension@ppanel.dev>
Date: Mon, 18 Aug 2025 02:54:00 -0400
Subject: [PATCH] fix(subscribe): rename variable for clarity and add special
 handling for Stash user agent

---
 internal/handler/subscribe.go              | 4 ++--
 internal/logic/subscribe/subscribeLogic.go | 5 +++++
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/internal/handler/subscribe.go b/internal/handler/subscribe.go
index 9cb82b6..08f2e91 100644
--- a/internal/handler/subscribe.go
+++ b/internal/handler/subscribe.go
@@ -28,9 +28,9 @@ func SubscribeHandler(svcCtx *svc.ServiceContext) func(c *gin.Context) {
 				c.Abort()
 				return
 			}
-			browserKeywords := strings.Split(svcCtx.Config.Subscribe.UserAgentList, "\n")
+			clientUserAgents := strings.Split(svcCtx.Config.Subscribe.UserAgentList, "\n")
 			var allow = false
-			for _, keyword := range browserKeywords {
+			for _, keyword := range clientUserAgents {
 				if strings.Contains(strings.ToLower(ua), strings.ToLower(keyword)) {
 					allow = true
 				}
diff --git a/internal/logic/subscribe/subscribeLogic.go b/internal/logic/subscribe/subscribeLogic.go
index 6f30fdb..a51ab90 100644
--- a/internal/logic/subscribe/subscribeLogic.go
+++ b/internal/logic/subscribe/subscribeLogic.go
@@ -53,7 +53,12 @@ func (l *SubscribeLogic) Handler(req *types.SubscribeRequest) (resp *types.Subsc
 		if item.IsDefault {
 			defaultApp = item
 		}
+
 		if strings.Contains(userAgent, u) {
+			// Special handling for Stash
+			if strings.Contains(userAgent, "stash") && !strings.Contains(u, "stash") {
+				continue
+			}
 			targetApp = item
 			break
 		}