-
Notifications
You must be signed in to change notification settings - Fork 537
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
Support ipv6 link-local addresses #470
Labels
Comments
jaymell
pushed a commit
that referenced
this issue
Sep 14, 2022
Should be fixed with #679. |
nonirosenfeldredis
pushed a commit
to nonirosenfeldredis/redis-rs
that referenced
this issue
Feb 13, 2023
socs
added a commit
to socs/redis-rs
that referenced
this issue
Mar 1, 2023
URL parsing fails when the host is a ipv6 link-local address due to zone identifiers being omitted by the url spec. https://url.spec.whatwg.org/#host-representation This fix resolves the issue by stripping the zone id from the url but adding it back into the host field during conversion to ConnectionInfo.
socs
added a commit
to socs/redis-rs
that referenced
this issue
Mar 1, 2023
URL parsing fails when the host is a ipv6 link-local address due to zone identifiers being omitted by the url spec. https://url.spec.whatwg.org/#host-representation This fix resolves the issue by stripping the zone id from the url but adding it back into the host field during conversion to ConnectionInfo.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
URL parsing fails on ipv6 link-local addresses, eg
This works with redis-cli
The limitation stems from rust-url library which follows a url spec that omits zone identifier on ipv6 ll addresses.
Perhaps a custom parser could be used by redis-rs, such as the one in redis-cli, so that ipv6 ll addresses is accepted.
The text was updated successfully, but these errors were encountered: