Skip to content

Commit

Permalink
Fixed issue in URL parsing
Browse files Browse the repository at this point in the history
Fixed "Broken URL parsing" issue mentioned in issue jquery-archive#8640 [details: 1) empty username or password are allowed, 2) colon in password is allowed]
  • Loading branch information
dryabov authored May 6, 2019
1 parent 1f0cec9 commit f3bd162
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion js/navigation/path.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ $.mobile.path = path = {
// [16]: ?msg=1234&type=unread
// [17]: #msg-content
//
urlParseRE: /^\s*(((([^:\/#\?]+:)?(?:(\/\/)((?:(([^:@\/#\?]+)(?:\:([^:@\/#\?]+))?)@)?(([^:\/#\?\]\[]+|\[[^\/\]@#?]+\])(?:\:([0-9]+))?))?)?)?((\/?(?:[^\/\?#]+\/+)*)([^\?#]*)))?(\?[^#]+)?)(#.*)?/,
urlParseRE: /^\s*(((([^:\/#\?]+:)?(?:(\/\/)((?:(([^:@\/#\?]*)(?:\:([^@\/#\?]*))?)@)?(([^:\/#\?\]\[]+|\[[^\/\]@#?]+\])(?:\:([0-9]+))?))?)?)?((\/?(?:[^\/\?#]+\/+)*)([^\?#]*)))?(\?[^#]+)?)(#.*)?/,

// Abstraction to address xss (Issue #4787) by removing the authority in
// browsers that auto-decode it. All references to location.href should be
Expand Down

0 comments on commit f3bd162

Please sign in to comment.