commit dec1c36b3a4a2856496fd2ed79735a11818e7e56
author Tim Duesterhus <tim@bastelstu.be> Mon May 10 17:28:26 2021 +0200
committer Willy Tarreau <w@1wt.eu> Tue May 11 17:24:32 2021 +0200
tree 4adc5e070623e4af66e07965745658e93bb7ef17
parent c9e05ab2de772a1c9a51a2f864001f0d23859836

MINOR: uri_normalizer: Add `fragment-encode` normalizer

This normalizer encodes '#' as '%23'.

See GitHub Issue #714.

reg-tests/http-rules/normalize_uri.vtc

diff --git a/reg-tests/http-rules/normalize_uri.vtc b/reg-tests/http-rules/normalize_uri.vtc
index 792bea5..7e2d749 100644
--- a/reg-tests/http-rules/normalize_uri.vtc
+++ b/reg-tests/http-rules/normalize_uri.vtc
@@ -8,7 +8,7 @@
 server s1 {
     rxreq
     txresp
-} -repeat 66 -start
+} -repeat 70 -start
 
 haproxy h1 -conf {
     global
@@ -137,6 +137,18 @@
 
         default_backend be
 
+    frontend fe_fragment_encode
+        bind "fd@${fe_fragment_encode}"
+
+        http-request set-var(txn.before) url
+        http-request normalize-uri fragment-encode
+        http-request set-var(txn.after) url
+
+        http-response add-header before  %[var(txn.before)]
+        http-response add-header after  %[var(txn.after)]
+
+        default_backend be
+
     backend be
         server s1 ${s1_addr}:${s1_port}
 
@@ -500,3 +512,25 @@
     expect resp.http.before == "*"
     expect resp.http.after == "*"
 } -run
+
+client c10 -connect ${h1_fe_fragment_encode_sock} {
+    txreq -url "/#foo"
+    rxresp
+    expect resp.http.before == "/#foo"
+    expect resp.http.after == "/%23foo"
+
+    txreq -url "/#foo/#foo"
+    rxresp
+    expect resp.http.before == "/#foo/#foo"
+    expect resp.http.after == "/%23foo/%23foo"
+
+    txreq -url "/%23foo"
+    rxresp
+    expect resp.http.before == "/%23foo"
+    expect resp.http.after == "/%23foo"
+
+    txreq -req OPTIONS -url "*"
+    rxresp
+    expect resp.http.before == "*"
+    expect resp.http.after == "*"
+} -run