Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

refactor js/lib/urlutil.js for safer url parsing #6098

Closed
diracdeltas opened this issue Dec 8, 2016 · 0 comments · Fixed by #10346
Closed

refactor js/lib/urlutil.js for safer url parsing #6098

diracdeltas opened this issue Dec 8, 2016 · 0 comments · Fixed by #10346
Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified refactoring release-notes/include security
Milestone
0.18.23 Hotfix

Comments

@diracdeltas
Copy link
Member

diracdeltas commented Dec 8, 2016 •
edited by luixxiul

Test plan

#10346 (comment)

it should try to use Chromium's URL parser whenever possible.
@diracdeltas diracdeltas mentioned this issue Aug 3, 2017
Closed
@alexwykoff alexwykoff added this to the 0.19.x (Beta Channel) milestone Aug 8, 2017
@diracdeltas diracdeltas mentioned this issue Aug 9, 2017
Merged
8 tasks
@bbondy bbondy modified the milestones: 0.18.x Hotfix, 0.19.x (Beta Channel) Aug 10, 2017
This was referenced Aug 10, 2017
Closed
Closed
Closed
Closed
This was referenced Aug 11, 2017
Closed
Closed
Closed
This was referenced Aug 16, 2017
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified refactoring release-notes/include security
Projects
None yet
Milestone
0.18.23 Hotfix
Development

Successfully merging a pull request may close this issue.

Fix URL parsing consistency issue brave/browser-laptop
7 participants
@diracdeltas @bbondy @alexwykoff @luixxiul @bsclifton @srirambv @LaurenWags