Skip to content

Conversation

@PixnBits
Copy link

This commit stops passing one domain's auth data to
another domain when passed as the relative path.

Fixes: #1435

This commit stops passing one domain's auth data to
another domain when passed as the relative path.

Fixes: nodejs#1435
@mscdex mscdex added the url Issues and PRs related to the legacy built-in url module. label Nov 18, 2015
@jasnell
Copy link
Member

jasnell commented Nov 18, 2015

duplication of : #1480

@PixnBits ... Unfortunately this change would need to be targeted at the master branch. We land changes there first and cherry pick back to the stable branches. We already have an existing open PR that's been stalled for a while on this. When I get a moment I'll try to do a compare/contrast on the two.

@PixnBits
Copy link
Author

Looks like you have more test cases which is a good thing.
Thanks for the info on how to contribute in the future!

@jasnell jasnell mentioned this pull request Nov 19, 2015
@PixnBits PixnBits closed this Nov 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

url Issues and PRs related to the legacy built-in url module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants