|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
[STR] 1. open a new tab and turn on Network Monitor 2. open http://www.example.org/#test in this tab 3. check the "Request URL" for the first HTTP request [Expected] The Request URL should be http://www.example.org/ without the URL fragment identifier (i.e. #test) because the real HTTP request doesn't include this part. [Actual] The Request URL shows http://www.example.org/#test
Has STR: --- → yes
Priority: -- → P3
We could call `new URL(link).origin` in HeadersPanel to return the correct Request URL https://developer.mozilla.org/en-US/docs/Web/API/URL/origin
Whiteboard: [good first bug][mentor-lang=zh][lang=js]
Hi Fred, I have created a mozreview-request for this issue and have also modified the tests a bit. Can you please take a look once?
Comment on attachment 8921573 [details] Bug 1251836 - Request URL should not include URL fragment identifier. https://reviewboard.mozilla.org/r/192590/#review197970 looks good, thanks!
Attachment #8921573 - Flags: review?(gasolin) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/fc0e0f8ae786 Request URL should not include URL fragment identifier. r=gasolin
Status: ASSIGNED → RESOLVED
Last Resolved: 9 months ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
You need to log in before you can comment on or make changes to this bug.