mirror of
				https://github.com/caddyserver/caddy.git
				synced 2025-10-30 18:22:49 -04:00 
			
		
		
		
	* fix: resolve http.request placeholders in header directive find operation - Skip regex compilation during provision when placeholders are detected - Compile regex at runtime after placeholder replacement - Preserves performance for static regexes while enabling dynamic placeholders - Fixes #7109 * test: add tests for placeholder detection in header replacements - Test containsPlaceholders function edge cases - Test provision skips compilation for dynamic regexes - Test end-to-end placeholder replacement functionality
		
			
				
	
	
		
			379 lines
		
	
	
		
			9.0 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			379 lines
		
	
	
		
			9.0 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2015 Matthew Holt and The Caddy Authors
 | |
| //
 | |
| // Licensed under the Apache License, Version 2.0 (the "License");
 | |
| // you may not use this file except in compliance with the License.
 | |
| // You may obtain a copy of the License at
 | |
| //
 | |
| //     http://www.apache.org/licenses/LICENSE-2.0
 | |
| //
 | |
| // Unless required by applicable law or agreed to in writing, software
 | |
| // distributed under the License is distributed on an "AS IS" BASIS,
 | |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 | |
| // See the License for the specific language governing permissions and
 | |
| // limitations under the License.
 | |
| 
 | |
| package headers
 | |
| 
 | |
| import (
 | |
| 	"context"
 | |
| 	"fmt"
 | |
| 	"net/http"
 | |
| 	"net/http/httptest"
 | |
| 	"reflect"
 | |
| 	"testing"
 | |
| 
 | |
| 	"github.com/caddyserver/caddy/v2"
 | |
| 	"github.com/caddyserver/caddy/v2/modules/caddyhttp"
 | |
| )
 | |
| 
 | |
| func TestHandler(t *testing.T) {
 | |
| 	for i, tc := range []struct {
 | |
| 		handler            Handler
 | |
| 		reqHeader          http.Header
 | |
| 		respHeader         http.Header
 | |
| 		respStatusCode     int
 | |
| 		expectedReqHeader  http.Header
 | |
| 		expectedRespHeader http.Header
 | |
| 	}{
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Request: &HeaderOps{
 | |
| 					Add: http.Header{
 | |
| 						"Expose-Secrets": []string{"always"},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			reqHeader: http.Header{
 | |
| 				"Expose-Secrets": []string{"i'm serious"},
 | |
| 			},
 | |
| 			expectedReqHeader: http.Header{
 | |
| 				"Expose-Secrets": []string{"i'm serious", "always"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Request: &HeaderOps{
 | |
| 					Set: http.Header{
 | |
| 						"Who-Wins": []string{"batman"},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			reqHeader: http.Header{
 | |
| 				"Who-Wins": []string{"joker"},
 | |
| 			},
 | |
| 			expectedReqHeader: http.Header{
 | |
| 				"Who-Wins": []string{"batman"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Request: &HeaderOps{
 | |
| 					Delete: []string{"Kick-Me"},
 | |
| 				},
 | |
| 			},
 | |
| 			reqHeader: http.Header{
 | |
| 				"Kick-Me": []string{"if you can"},
 | |
| 				"Keep-Me": []string{"i swear i'm innocent"},
 | |
| 			},
 | |
| 			expectedReqHeader: http.Header{
 | |
| 				"Keep-Me": []string{"i swear i'm innocent"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Request: &HeaderOps{
 | |
| 					Delete: []string{
 | |
| 						"*-suffix",
 | |
| 						"prefix-*",
 | |
| 						"*_*",
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			reqHeader: http.Header{
 | |
| 				"Header-Suffix": []string{"lalala"},
 | |
| 				"Prefix-Test":   []string{"asdf"},
 | |
| 				"Host_Header":   []string{"silly django... sigh"}, // see issue #4830
 | |
| 				"Keep-Me":       []string{"foofoofoo"},
 | |
| 			},
 | |
| 			expectedReqHeader: http.Header{
 | |
| 				"Keep-Me": []string{"foofoofoo"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Request: &HeaderOps{
 | |
| 					Replace: map[string][]Replacement{
 | |
| 						"Best-Server": {
 | |
| 							Replacement{
 | |
| 								Search:  "NGINX",
 | |
| 								Replace: "the Caddy web server",
 | |
| 							},
 | |
| 							Replacement{
 | |
| 								SearchRegexp: `Apache(\d+)`,
 | |
| 								Replace:      "Caddy",
 | |
| 							},
 | |
| 						},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			reqHeader: http.Header{
 | |
| 				"Best-Server": []string{"it's NGINX, undoubtedly", "I love Apache2"},
 | |
| 			},
 | |
| 			expectedReqHeader: http.Header{
 | |
| 				"Best-Server": []string{"it's the Caddy web server, undoubtedly", "I love Caddy"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Response: &RespHeaderOps{
 | |
| 					Require: &caddyhttp.ResponseMatcher{
 | |
| 						Headers: http.Header{
 | |
| 							"Cache-Control": nil,
 | |
| 						},
 | |
| 					},
 | |
| 					HeaderOps: &HeaderOps{
 | |
| 						Add: http.Header{
 | |
| 							"Cache-Control": []string{"no-cache"},
 | |
| 						},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			respHeader: http.Header{},
 | |
| 			expectedRespHeader: http.Header{
 | |
| 				"Cache-Control": []string{"no-cache"},
 | |
| 			},
 | |
| 		},
 | |
| 		{ // same as above, but checks that response headers are left alone when "Require" conditions are unmet
 | |
| 			handler: Handler{
 | |
| 				Response: &RespHeaderOps{
 | |
| 					Require: &caddyhttp.ResponseMatcher{
 | |
| 						Headers: http.Header{
 | |
| 							"Cache-Control": nil,
 | |
| 						},
 | |
| 					},
 | |
| 					HeaderOps: &HeaderOps{
 | |
| 						Add: http.Header{
 | |
| 							"Cache-Control": []string{"no-cache"},
 | |
| 						},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			respHeader: http.Header{
 | |
| 				"Cache-Control": []string{"something"},
 | |
| 			},
 | |
| 			expectedRespHeader: http.Header{
 | |
| 				"Cache-Control": []string{"something"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Response: &RespHeaderOps{
 | |
| 					Require: &caddyhttp.ResponseMatcher{
 | |
| 						Headers: http.Header{
 | |
| 							"Cache-Control": []string{"no-cache"},
 | |
| 						},
 | |
| 					},
 | |
| 					HeaderOps: &HeaderOps{
 | |
| 						Delete: []string{"Cache-Control"},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			respHeader: http.Header{
 | |
| 				"Cache-Control": []string{"no-cache"},
 | |
| 			},
 | |
| 			expectedRespHeader: http.Header{},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Response: &RespHeaderOps{
 | |
| 					Require: &caddyhttp.ResponseMatcher{
 | |
| 						StatusCode: []int{5},
 | |
| 					},
 | |
| 					HeaderOps: &HeaderOps{
 | |
| 						Add: http.Header{
 | |
| 							"Fail-5xx": []string{"true"},
 | |
| 						},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			respStatusCode: 503,
 | |
| 			respHeader:     http.Header{},
 | |
| 			expectedRespHeader: http.Header{
 | |
| 				"Fail-5xx": []string{"true"},
 | |
| 			},
 | |
| 		},
 | |
| 		{
 | |
| 			handler: Handler{
 | |
| 				Request: &HeaderOps{
 | |
| 					Replace: map[string][]Replacement{
 | |
| 						"Case-Insensitive": {
 | |
| 							Replacement{
 | |
| 								Search:  "issue4330",
 | |
| 								Replace: "issue #4330",
 | |
| 							},
 | |
| 						},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 			reqHeader: http.Header{
 | |
| 				"case-insensitive": []string{"issue4330"},
 | |
| 				"Other-Header":     []string{"issue4330"},
 | |
| 			},
 | |
| 			expectedReqHeader: http.Header{
 | |
| 				"case-insensitive": []string{"issue #4330"},
 | |
| 				"Other-Header":     []string{"issue4330"},
 | |
| 			},
 | |
| 		},
 | |
| 	} {
 | |
| 		rr := httptest.NewRecorder()
 | |
| 
 | |
| 		req := &http.Request{Header: tc.reqHeader}
 | |
| 		repl := caddy.NewReplacer()
 | |
| 		ctx := context.WithValue(req.Context(), caddy.ReplacerCtxKey, repl)
 | |
| 		req = req.WithContext(ctx)
 | |
| 
 | |
| 		tc.handler.Provision(caddy.Context{})
 | |
| 
 | |
| 		next := nextHandler(func(w http.ResponseWriter, r *http.Request) error {
 | |
| 			for k, hdrs := range tc.respHeader {
 | |
| 				for _, v := range hdrs {
 | |
| 					w.Header().Add(k, v)
 | |
| 				}
 | |
| 			}
 | |
| 
 | |
| 			status := 200
 | |
| 			if tc.respStatusCode != 0 {
 | |
| 				status = tc.respStatusCode
 | |
| 			}
 | |
| 			w.WriteHeader(status)
 | |
| 
 | |
| 			if tc.expectedReqHeader != nil && !reflect.DeepEqual(r.Header, tc.expectedReqHeader) {
 | |
| 				return fmt.Errorf("expected request header %v, got %v", tc.expectedReqHeader, r.Header)
 | |
| 			}
 | |
| 
 | |
| 			return nil
 | |
| 		})
 | |
| 
 | |
| 		if err := tc.handler.ServeHTTP(rr, req, next); err != nil {
 | |
| 			t.Errorf("Test %d: %v", i, err)
 | |
| 			continue
 | |
| 		}
 | |
| 
 | |
| 		actual := rr.Header()
 | |
| 		if tc.expectedRespHeader != nil && !reflect.DeepEqual(actual, tc.expectedRespHeader) {
 | |
| 			t.Errorf("Test %d: expected response header %v, got %v", i, tc.expectedRespHeader, actual)
 | |
| 			continue
 | |
| 		}
 | |
| 	}
 | |
| }
 | |
| 
 | |
| type nextHandler func(http.ResponseWriter, *http.Request) error
 | |
| 
 | |
| func (f nextHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) error {
 | |
| 	return f(w, r)
 | |
| }
 | |
| 
 | |
| func TestContainsPlaceholders(t *testing.T) {
 | |
| 	for i, tc := range []struct {
 | |
| 		input    string
 | |
| 		expected bool
 | |
| 	}{
 | |
| 		{"static", false},
 | |
| 		{"{placeholder}", true},
 | |
| 		{"prefix-{placeholder}-suffix", true},
 | |
| 		{"{}", false},
 | |
| 		{"no-braces", false},
 | |
| 		{"{unclosed", false},
 | |
| 		{"unopened}", false},
 | |
| 	} {
 | |
| 		actual := containsPlaceholders(tc.input)
 | |
| 		if actual != tc.expected {
 | |
| 			t.Errorf("Test %d: containsPlaceholders(%q) = %v, expected %v", i, tc.input, actual, tc.expected)
 | |
| 		}
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestHeaderProvisionSkipsPlaceholders(t *testing.T) {
 | |
| 	ops := &HeaderOps{
 | |
| 		Replace: map[string][]Replacement{
 | |
| 			"Static": {
 | |
| 				Replacement{SearchRegexp: ":443", Replace: "STATIC"},
 | |
| 			},
 | |
| 			"Dynamic": {
 | |
| 				Replacement{SearchRegexp: ":{http.request.local.port}", Replace: "DYNAMIC"},
 | |
| 			},
 | |
| 		},
 | |
| 	}
 | |
| 
 | |
| 	err := ops.Provision(caddy.Context{})
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("Provision failed: %v", err)
 | |
| 	}
 | |
| 
 | |
| 	// Static regex should be precompiled
 | |
| 	if ops.Replace["Static"][0].re == nil {
 | |
| 		t.Error("Expected static regex to be precompiled")
 | |
| 	}
 | |
| 
 | |
| 	// Dynamic regex with placeholder should not be precompiled
 | |
| 	if ops.Replace["Dynamic"][0].re != nil {
 | |
| 		t.Error("Expected dynamic regex with placeholder to not be precompiled")
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestPlaceholderInSearchRegexp(t *testing.T) {
 | |
| 	handler := Handler{
 | |
| 		Response: &RespHeaderOps{
 | |
| 			HeaderOps: &HeaderOps{
 | |
| 				Replace: map[string][]Replacement{
 | |
| 					"Test-Header": {
 | |
| 						Replacement{
 | |
| 							SearchRegexp: ":{http.request.local.port}",
 | |
| 							Replace:      "PLACEHOLDER-WORKS",
 | |
| 						},
 | |
| 					},
 | |
| 				},
 | |
| 			},
 | |
| 		},
 | |
| 	}
 | |
| 
 | |
| 	// Provision the handler
 | |
| 	err := handler.Provision(caddy.Context{})
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("Provision failed: %v", err)
 | |
| 	}
 | |
| 
 | |
| 	replacement := handler.Response.HeaderOps.Replace["Test-Header"][0]
 | |
| 	t.Logf("After provision - SearchRegexp: %q, re: %v", replacement.SearchRegexp, replacement.re)
 | |
| 
 | |
| 	rr := httptest.NewRecorder()
 | |
| 
 | |
| 	req := httptest.NewRequest("GET", "http://localhost:443/", nil)
 | |
| 	repl := caddy.NewReplacer()
 | |
| 	repl.Set("http.request.local.port", "443")
 | |
| 
 | |
| 	ctx := context.WithValue(req.Context(), caddy.ReplacerCtxKey, repl)
 | |
| 	req = req.WithContext(ctx)
 | |
| 
 | |
| 	rr.Header().Set("Test-Header", "prefix:443suffix")
 | |
| 	t.Logf("Initial header: %v", rr.Header())
 | |
| 
 | |
| 	next := nextHandler(func(w http.ResponseWriter, r *http.Request) error {
 | |
| 		w.WriteHeader(200)
 | |
| 		return nil
 | |
| 	})
 | |
| 
 | |
| 	err = handler.ServeHTTP(rr, req, next)
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("ServeHTTP failed: %v", err)
 | |
| 	}
 | |
| 
 | |
| 	t.Logf("Final header: %v", rr.Header())
 | |
| 
 | |
| 	result := rr.Header().Get("Test-Header")
 | |
| 	expected := "prefixPLACEHOLDER-WORKSsuffix"
 | |
| 	if result != expected {
 | |
| 		t.Errorf("Expected header value %q, got %q", expected, result)
 | |
| 	}
 | |
| }
 |