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

urlapi: fix parsing ipv6 with zone index #3411

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@danielgustafsson
Copy link
Member

danielgustafsson commented Dec 26, 2018

The previous fix for parsing IPv6 URLs with a zone index was a paddle short for URLs without an explicit port. Fix for this case and add a unit test case.

This bug was highlighted by issue #3408, and while it's not the full fix for the problem there it is an isolated bug that should be fixed.

urlapi: fix parsing ipv6 with zone index
The previous fix for parsing IPv6 URLs with a zone index was a paddle
short for URLs without an explicit port. Fix for this case and add a
unit test case.

This bug was highlighted by issue #3408, and while it's not the full
fix for the problem there it is an isolated bug that should be fixed.
@bagder

bagder approved these changes Dec 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment