Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

url: don't keep auth information when changing host #3893

Closed
wants to merge 1 commit into from

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