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.

Sign up for GitHub

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

Jump to bottom

Fix bracketed hostnames #236

Merged
merged 5 commits into from Dec 30, 2023
Merged

Fix bracketed hostnames #236

merged 5 commits into from Dec 30, 2023

Conversation

RubenVerborgh
Copy link
Collaborator

@RubenVerborgh RubenVerborgh commented Dec 29, 2023 •
edited

Fixes #235

@RubenVerborgh RubenVerborgh self-assigned this Dec 29, 2023
@RubenVerborgh RubenVerborgh marked this pull request as draft December 29, 2023 10:17
@RubenVerborgh RubenVerborgh mentioned this pull request Dec 29, 2023
Closed
@RubenVerborgh
Copy link
Collaborator Author

Note to self: need to remove all url.parse instances where new URL is available.

@RubenVerborgh RubenVerborgh marked this pull request as ready for review December 30, 2023 17:22
@RubenVerborgh RubenVerborgh merged commit 7a6567e into main Dec 30, 2023
24 checks passed
@github-actions github-actions bot mentioned this pull request Jan 1, 2024
Closed
@samc621
Copy link

samc621 commented Jan 12, 2024 •
edited

Hi @RubenVerborgh, I have a small fix for the changes in 05629af from this PR in #241.

This was referenced Jan 18, 2024
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@RubenVerborgh RubenVerborgh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Security Report] hostname spoofing via url.parse in follow-redirects
4 participants
@RubenVerborgh @samc621 @rhuankarlus @Renanluizssx