works

Author: evgeniy-scherbina

go.mod

@@ -2,12 +2,16 @@ module github.com/coder/boundary
 
 go 1.24
 
-require github.com/coder/serpent v0.10.0
+require (
+	github.com/coder/serpent v0.10.0
+	github.com/stretchr/testify v1.8.4
+)
 
 require (
 	cdr.dev/slog v1.6.2-0.20240126064726-20367d4aede6 // indirect
 	github.com/aymanbagabas/go-osc52/v2 v2.0.1 // indirect
 	github.com/coder/pretty v0.0.0-20230908205945-e89ba86370e0 // indirect
+	github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc // indirect
 	github.com/hashicorp/errwrap v1.1.0 // indirect
 	github.com/hashicorp/go-multierror v1.1.1 // indirect
 	github.com/kr/text v0.2.0 // indirect
@@ -18,6 +22,7 @@ require (
 	github.com/muesli/termenv v0.15.2 // indirect
 	github.com/pion/transport/v2 v2.0.0 // indirect
 	github.com/pion/udp v0.1.4 // indirect
+	github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
 	github.com/rivo/uniseg v0.4.4 // indirect
 	github.com/spf13/pflag v1.0.5 // indirect
 	go.opentelemetry.io/otel v1.19.0 // indirect

proxy/proxy_test.go

@@ -3,12 +3,16 @@ package proxy
 import (
 	"context"
 	"crypto/tls"
-	"io"
+	"fmt"
+	"io/ioutil"
 	"log/slog"
 	"net/http"
+	"os"
 	"testing"
 	"time"
 
+	"github.com/stretchr/testify/require"
+
 	"github.com/coder/boundary/audit"
 	"github.com/coder/boundary/rules"
 )
@@ -23,8 +27,8 @@ func (m *mockAuditor) AuditRequest(req audit.Request) {
 // TestProxyServerBasicHTTP tests basic HTTP request handling
 func TestProxyServerBasicHTTP(t *testing.T) {
 	// Create test logger
-	logger := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{
-		Level: slog.LevelError, // Reduce noise during testing
+	logger := slog.New(slog.NewTextHandler(os.Stderr, &slog.HandlerOptions{
+		Level: slog.LevelDebug, // Reduce noise during testing
 	}))
 
 	// Create test rules (allow all for testing)
@@ -46,7 +50,7 @@ func TestProxyServerBasicHTTP(t *testing.T) {
 
 	// Create proxy server
 	server := NewProxyServer(Config{
-		HTTPPort:   0, // Use random port
+		HTTPPort:   8080, // Use random port
 		RuleEngine: ruleEngine,
 		Auditor:    auditor,
 		Logger:     logger,
@@ -79,22 +83,33 @@ func TestProxyServerBasicHTTP(t *testing.T) {
 		}
 
 		// Make request to proxy
-		req, err := http.NewRequest("GET", "http://localhost:8080/test", nil)
+		req, err := http.NewRequest("GET", "http://localhost:8080", nil)
 		if err != nil {
 			t.Fatalf("Failed to create request: %v", err)
 		}
+		// Override the Host header to coder.com
+		req.Host = "coder.com"
+		//req.Header.Set("Host", "coder.com")
 
 		// Set Host header (important for URL parsing)
-		req.Host = "localhost:8080"
+		//req.Host = "localhost:8080"
 
 		// Make the request
 		resp, err := client.Do(req)
 		if err != nil {
 			t.Logf("Request failed (expected for proxy without target): %v", err)
 			// This is expected since we're not forwarding to a real target
-		} else {
-			resp.Body.Close()
 		}
+		//else {
+		//	resp.Body.Close()
+		//}
+		fmt.Printf("err: %v\n", err)
+
+		body, err := ioutil.ReadAll(resp.Body)
+		require.NoError(t, err)
+		fmt.Printf("body: %s\n", body)
+
+		resp.Body.Close()
 	})
 
 	// Test CONNECT request