Pull request 2614: AGDNS-3799-h2c-vuln-test

Squashed commit of the following:

commit 43c0748
Merge: 1a9c78d 35f9101
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 23 17:21:12 2026 +0300

    Merge branch 'master' into AGDNS-3799-h2c-vuln-test

commit 1a9c78d
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 23 17:09:55 2026 +0300

    home: imp docs

commit 729d41a
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 23 14:18:26 2026 +0300

    home: rm temp logs

commit 4ad68ef
Merge: 4e33bab b08e587
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 23 14:04:37 2026 +0300

    Merge branch 'master' into AGDNS-3799-h2c-vuln-test

commit 4e33bab
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 23 13:41:13 2026 +0300

    home: print stack

commit 7bcd543
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 23 13:33:06 2026 +0300

    home: add temp log

commit cb21e81
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Tue Apr 21 18:10:30 2026 +0300

    home: enable h2c test logs

commit f0785d9
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 16 17:59:40 2026 +0300

    home: rm querylog, imp code

commit 98fea60
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Wed Apr 15 19:00:38 2026 +0300

    home: use static files for healthcheck

commit b01973e
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Wed Apr 15 11:56:26 2026 +0300

    home: disable conn reuse

commit 4b7ce72
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Tue Apr 14 23:42:12 2026 +0300

    home: increase timeout

commit 7ffbc64
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Tue Apr 14 23:27:02 2026 +0300

    home: use /control/status for healthcheck

commit bc45ee8
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Tue Apr 14 18:39:28 2026 +0300

    web: fix req body, increase timeouts

commit 9130442
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 9 20:30:30 2026 +0300

    home: fix ci, imp style

commit be70c8f
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Mon Apr 6 15:51:42 2026 +0300

    home: imp test

commit 0bff130
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Thu Apr 2 19:08:44 2026 +0300

    home: impl upgrade requests

commit 483ed1f
Author: f.setrakov <f.setrakov@adguard.com>
Date:   Fri Mar 27 13:07:07 2026 +0300

    home: h2c vuln test draft

Author: masterkusok

internal/home/clientshttp_internal_test.go

@@ -11,7 +11,6 @@ import (
 	"net/url"
 	"slices"
 	"testing"
-	"time"
 
 	"github.com/AdguardTeam/AdGuardHome/internal/client"
 	"github.com/AdguardTeam/AdGuardHome/internal/filtering"
@@ -22,9 +21,6 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-// testTimeout is the common timeout for tests and contexts.
-const testTimeout = 1 * time.Second
-
 const (
 	testClientIP1 = "1.1.1.1"
 	testClientIP2 = "2.2.2.2"

internal/home/home_internal_test.go

@@ -5,6 +5,7 @@ import (
 	"net/http"
 	"net/netip"
 	"testing"
+	"time"
 
 	"github.com/AdguardTeam/AdGuardHome/internal/agh"
 	"github.com/AdguardTeam/AdGuardHome/internal/aghhttp"
@@ -14,6 +15,9 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+// testTimeout is the common timeout for tests and contexts.
+const testTimeout = 1 * time.Second
+
 // testLogger is a common logger for tests.
 var testLogger = slogutil.NewDiscardLogger()
 
@@ -52,7 +56,38 @@ func newTestWeb(
 	return web
 }
 
+// storeGlobals is a test helper function that saves global variables and
+// restores them once the test is complete.
+//
+// The global variables are:
+//   - [config]
+//   - [glFilePrefix]
+//   - [globalContext.clients.storage]
+//   - [globalContext.dnsServer]
+//   - [globalContext.web]
+//
+// TODO(s.chzhen):  Remove this once the TLS manager no longer accesses global
+// variables.  Make tests that use this helper concurrent.
+func storeGlobals(tb testing.TB) {
+	tb.Helper()
+
+	prevConfig := config
+	prefGLFilePrefix := glFilePrefix
+	storage := globalContext.clients.storage
+	dnsServer := globalContext.dnsServer
+	web := globalContext.web
+
+	tb.Cleanup(func() {
+		config = prevConfig
+		glFilePrefix = prefGLFilePrefix
+		globalContext.clients.storage = storage
+		globalContext.dnsServer = dnsServer
+		globalContext.web = web
+	})
+}
+
 func TestMain(m *testing.M) {
 	initCmdLineOpts()
+
 	testutil.DiscardLogOutput(m)
 }

internal/home/tls_internal_test.go

@@ -133,39 +133,6 @@ func TestValidateCertificates(t *testing.T) {
 	})
 }
 
-// storeGlobals is a test helper function that saves global variables and
-// restores them once the test is complete.
-//
-// The global variables are:
-//   - [config]
-//   - [glFilePrefix]
-//   - [globalContext.auth]
-//   - [globalContext.clients.storage]
-//   - [globalContext.dnsServer]
-//   - [globalContext.firstRun]
-//   - [globalContext.mux]
-//   - [globalContext.web]
-//
-// TODO(s.chzhen):  Remove this once the TLS manager no longer accesses global
-// variables.  Make tests that use this helper concurrent.
-func storeGlobals(tb testing.TB) {
-	tb.Helper()
-
-	prevConfig := config
-	prefGLFilePrefix := glFilePrefix
-	storage := globalContext.clients.storage
-	dnsServer := globalContext.dnsServer
-	web := globalContext.web
-
-	tb.Cleanup(func() {
-		config = prevConfig
-		glFilePrefix = prefGLFilePrefix
-		globalContext.clients.storage = storage
-		globalContext.dnsServer = dnsServer
-		globalContext.web = web
-	})
-}
-
 // newCertWithoutIP generates a CA certificate, a leaf certificate without an IP
 // address, and the PEM-encoded leaf private key.
 func newCertWithoutIP(tb testing.TB) (