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

Parsing URLs with IPv6 host #12

Open
aryszka opened this issue Aug 24, 2018 · 9 comments · May be fixed by #35
Open

Parsing URLs with IPv6 host #12

aryszka opened this issue Aug 24, 2018 · 9 comments · May be fixed by #35

Comments

@aryszka
Copy link

aryszka commented Aug 24, 2018

I tried to use URLs with IPv6 host for a Browser.application and the navigation didn't work. The log message in the console directed me to this page: https://github.com/elm/core/blob/1.0.0/hints/1.md . When tried the same with IPv4 or a domain name, then it worked.

The URLs that I tried:

  • http://[::]:7272/ui/
  • http://[::1]:7272/ui/

Is this a bug or just the feature is not supported currently?

@4onen
Copy link

4onen commented May 20, 2019

Appears to be a bug. Line 167 of url/Url.elm indexes the first : in the host of the URL and, if it detects more than one :, assumes the URL is invalid. Should I write a PR that checks for this IPv6 shape before failing?

@jeffrey4l
Copy link

i am using alertmanager in a pure ipv6 environment. and got the same exact issue. if i use a domain name instead of ipv6 address, it works.

@AlexanderEkdahl AlexanderEkdahl linked a pull request Oct 5, 2019 that will close this issue
@AlexanderEkdahl
Copy link

AlexanderEkdahl commented Oct 5, 2019

I also ran into this problem. PR: #35

@venkatbvc
Copy link

we are having this issue as well

@venkatbvc
Copy link

@evancz Can you please look at the fix provided by @AlexanderEkdahl and if all is fine merge the fix? We have been waiting for this for long time.

@sriharshabm
Copy link

We are also facing issue, If changes provided by @AlexanderEkdahl is fine then can it be merged?

@sriharshabm
Copy link

Could you please update on this ticket

@sriharshabm
Copy link

We are also facing issue, If changes provided by @AlexanderEkdahl is fine then can it be merged?

Could you pls comment on this

@SarthakSahu
Copy link

@evancz @MrL1605 Could you please help on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants