diff --git a/internal/sub/external_hwid_test.go b/internal/sub/external_hwid_test.go new file mode 100644 index 000000000..ad0e210be --- /dev/null +++ b/internal/sub/external_hwid_test.go @@ -0,0 +1,67 @@ +package sub + +import ( + "net/http" + "net/http/httptest" + "path/filepath" + "testing" + + "github.com/mhsanaei/3x-ui/v3/internal/database" + "github.com/mhsanaei/3x-ui/v3/internal/database/model" +) + +// #6559: the Master panel must send a stable X-HWID when fetching external +// subscriptions, otherwise an HWID-limited donor answers 404. +func TestServerHwidStableAcrossCalls(t *testing.T) { + if err := database.InitDB(filepath.Join(t.TempDir(), "x-ui.db")); err != nil { + t.Fatalf("InitDB: %v", err) + } + t.Cleanup(func() { _ = database.CloseDB() }) + + first := serverHwid() + if first == "" { + t.Fatal("serverHwid returned empty") + } + + second := serverHwid() + if second != first { + t.Fatalf("hwid not stable: %q vs %q", first, second) + } + + var row model.Setting + if err := database.GetDB().Where("key = ?", serverHwidKey).First(&row).Error; err != nil { + t.Fatalf("hwid not persisted: %v", err) + } + if row.Value != first { + t.Fatalf("persisted hwid %q != returned %q", row.Value, first) + } +} + +// The fetch must carry the stable id so an HWID-limited donor lets it through. +func TestFetchSendsStableHwid(t *testing.T) { + if err := database.InitDB(filepath.Join(t.TempDir(), "x-ui.db")); err != nil { + t.Fatalf("InitDB: %v", err) + } + t.Cleanup(func() { _ = database.CloseDB() }) + + var gotHwid string + srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { + gotHwid = r.Header.Get("X-HWID") + _, _ = w.Write([]byte("vless://uuid@host:443?security=none#x")) + })) + defer srv.Close() + + res := fetchSubscriptionLinks(srv.URL) + if res.err != nil { + t.Fatalf("fetch: %v", res.err) + } + if len(res.links) != 1 { + t.Fatalf("links = %v", res.links) + } + if gotHwid == "" { + t.Fatal("X-HWID header missing on fetch") + } + if gotHwid != serverHwid() { + t.Fatalf("sent %q != stable %q", gotHwid, serverHwid()) + } +} diff --git a/internal/sub/external_subscription.go b/internal/sub/external_subscription.go index 862cbb496..e5aeb8682 100644 --- a/internal/sub/external_subscription.go +++ b/internal/sub/external_subscription.go @@ -9,15 +9,15 @@ import ( "sync" "time" + "github.com/google/uuid" + "github.com/mhsanaei/3x-ui/v3/internal/database" "github.com/mhsanaei/3x-ui/v3/internal/database/model" "github.com/mhsanaei/3x-ui/v3/internal/logger" ) -// External subscription fetching: a "subscription" external link is a remote -// URL whose body is a (often base64-encoded) newline list of share links. We -// fetch it on demand, cache the decoded links briefly, and bound the request -// with a short timeout so a slow/dead provider can't stall a client's sub. +// External subscription fetching: a remote URL whose body is a share-link +// list. Fetches are cached briefly and bounded so a dead provider can't stall. const ( subscriptionCacheTTL = 5 * time.Minute @@ -150,6 +150,10 @@ func doFetchSubscriptionLinks(rawURL string) ([]string, error) { } // Some providers gate the link body on a known client User-Agent. req.Header.Set("User-Agent", "v2rayNG/1.8.5") + // A 3x-ui donor with an HWID limit answers 404 when the header is empty (#6559). + if hwid := serverHwid(); hwid != "" { + req.Header.Set("X-HWID", hwid) + } resp, err := subscriptionHTTPClient.Do(req) if err != nil { return nil, err @@ -173,6 +177,41 @@ var ( errSubscriptionBodyTooLarge = &subError{"subscription response body exceeds size limit"} ) +// serverHwidKey is the settings row holding this panel's stable identity +// for outbound external-subscription fetches. +const serverHwidKey = "externalSubHwid" + +// serverHwidMu serializes first-time creation: without it, concurrent first +// fetches of different URLs each mint and persist their own UUID. +var serverHwidMu sync.Mutex + +// serverHwid returns a stable per-installation id, creating and persisting +// it on first use. Empty means the DB is unreachable: send no header then. +func serverHwid() string { + serverHwidMu.Lock() + defer serverHwidMu.Unlock() + db := database.GetDB() + if db == nil { + return "" + } + var row model.Setting + if err := db.Where("key = ?", serverHwidKey).First(&row).Error; err == nil { + if strings.TrimSpace(row.Value) != "" { + return strings.TrimSpace(row.Value) + } + } + hwid := "3x-ui-server-" + uuid.NewString() + row = model.Setting{Key: serverHwidKey, Value: hwid} + if err := db.Where(model.Setting{Key: serverHwidKey}).FirstOrCreate(&row).Error; err != nil { + logger.Warningf("sub: persisting server hwid failed: %v", err) + return "" + } + if strings.TrimSpace(row.Value) == "" { + return hwid + } + return strings.TrimSpace(row.Value) +} + type subError struct{ msg string } func (e *subError) Error() string { return e.msg }