BBlack has uploaded a new change for review.
https://gerrit.wikimedia.org/r/274086
Change subject: normalize_path: stop on fragment marker
......................................................................
normalize_path: stop on fragment marker
The fragment part shouldn't be sent to servers, but we should
support it like this anyways in case of broken clients.
Bug: T127387
Change-Id: Id719c53e2931fff67a124ab084bc21aa21be1122
---
M templates/varnish/normalize_path.inc.vcl.erb
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/86/274086/1
diff --git a/templates/varnish/normalize_path.inc.vcl.erb
b/templates/varnish/normalize_path.inc.vcl.erb
index e145508..baa4fb2 100644
--- a/templates/varnish/normalize_path.inc.vcl.erb
+++ b/templates/varnish/normalize_path.inc.vcl.erb
@@ -54,8 +54,8 @@
} else {
destBuffer[outPos++] = url[i];
}
- } else if (url[i] == '?') {
- /* Reached the query part. Just copy the rest
of the URL
+ } else if (url[i] == '?' || url[i] = '#') {
+ /* Reached the query/frag part. Just copy the
rest of the URL
* to the destination.
*/
memcpy(destBuffer + outPos, url + i,
sizeof(char) * (urlLength - i));
--
To view, visit https://gerrit.wikimedia.org/r/274086
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id719c53e2931fff67a124ab084bc21aa21be1122
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BBlack <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
