255924 2023-04-25 08:12:04 -0700 URI Fragments Cached After 301 Redirect 2023-07-10 10:04:29 -0700 1 1 1 Unclassified WebKit Page Loading Safari 16 All Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=257327 InRadar P2 Major --- 1 zachshaw1 brandonstewart achristensen beidson brandonstewart bugs-noreply cdumez koivisto mcatanzaro saralkaushik simon.khalil webkit-bug-importer youennf oldest_to_newest 1951052 0 zachshaw1 2023-04-25 08:12:04 -0700 Version: Safari 16.4 (macOS & iOS) When Safari is redirected by a 301 with URI fragments, for example site.com/page#key=123 -> www.site.com/page#key=123, any subsequent page hits after the 301, Safari will use the initial cached URI fragments. This isn't happening on any other browser, nor is this happening on Safari 16.3 the new URI fragments are correctly picked up. Disable Caches enabled & private browsing both work fine as well. 1951056 1 simon.khalil 2023-04-25 08:18:39 -0700 I'm having this issue too. Strangely after a 301 the URL fragment is picking up cached values instead of the new fragment values. Very odd behaviour 1951229 2 webkit-bug-importer 2023-04-25 18:38:40 -0700 <rdar://problem/108535944> 1957971 3 brandonstewart 2023-05-25 11:48:29 -0700 Was doing some testing and was able to reproduce back on Safari 16.2 and a recent version of Safari. Testing Redirect Server from http.server import HTTPServer, BaseHTTPRequestHandler class Redirect(BaseHTTPRequestHandler): def do_GET(self): print("hello") self.send_response(301) self.send_header('Location', 'http://localhost:8000') self.send_header('Referrer-Policy', 'same-origin') self.end_headers() HTTPServer(('', 10000), Redirect).serve_forever() ################## Server to redirect to python3 -m http.server --bind 127.0.0.1 --directory . 8000 index.html <!DOCTYPE html> <html> <p>Hello World</p> </html> URL Go to localhost:10000/#key=foo Then go to localhost:10000/#key=bar I noticed it works correctly on the second time you visit the redirect. URL Enter -> URL Result localhost:10000/#key=foo -> localhost:8000/#key=foo localhost:10000/#key=bar -> localhost:8000/#key=foo localhost:10000/#key=bar -> localhost:8000/#key=bar 1958874 4 cdumez 2023-05-30 13:49:05 -0700 *** Bug 257327 has been marked as a duplicate of this bug. *** 1958875 5 cdumez 2023-05-30 13:50:26 -0700 Seem information from dupe at https://bugs.webkit.org/show_bug.cgi?id=257327#c2 to help fix this. Odds are we simply want to stop caching redirects when the request URL contains a fragment. 1958876 6 cdumez 2023-05-30 13:53:15 -0700 (In reply to Chris Dumez from comment #5) > Seem information from dupe at > https://bugs.webkit.org/show_bug.cgi?id=257327#c2 to help fix this. > > Odds are we simply want to stop caching redirects when the request URL > contains a fragment. It is not OK to navigate to https://www.monzo.com/#foo=bar, it is not OK to cache en entry like this: ``` https://www.monzo.com (because we drop fragment from cache keys) -> https://monzo.com/#foo=bar ``` Because this cached redirect would get reused for every load of https://www.monzo.com, no matter the fragment identifier. 1958903 7 brandonstewart 2023-05-30 16:26:28 -0700 Pull request: https://github.com/WebKit/WebKit/pull/14514 1965966 8 ews-feeder 2023-07-10 10:04:26 -0700 Committed 265906@main (205444531ae8): <https://commits.webkit.org/265906@main> Reviewed commits have been landed. Closing PR #14514 and removing active labels.