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

urlapi: normalize the IPv6 address #15143

Closed
wants to merge 1 commit into from
Closed

urlapi: normalize the IPv6 address #15143

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Oct 3, 2024

As the parsing and address "regeneration" are done anyway, we might as well use the updated version in the result and thereby A) get a normalized (and lower cased) version of the address and B) avoid a strcpy().

Updated test 1560 to verify.

As the parsing and address "regeneration" are done anyway, we might as
well use the updated version in the result and thereby A) get a
normalized (and lower cased) version of the address and B) avoid a
strcpy().

Updated test 1560 to verify.
@bagder bagder added the URL label Oct 3, 2024
@github-actions github-actions bot added the tests label Oct 3, 2024
@bagder bagder closed this in 566a6d7 Oct 3, 2024
@bagder bagder deleted the bagder/urlapi-ipv6 branch October 3, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant