url icon indicating copy to clipboard operation
url copied to clipboard

Parsing URLs with IPv6 host

Open aryszka opened this issue 6 years ago • 9 comments

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?

aryszka avatar Aug 24 '18 16:08 aryszka

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?

4onen avatar May 20 '19 18:05 4onen

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.

jeffrey4l avatar Jul 10 '19 07:07 jeffrey4l

I also ran into this problem. PR: https://github.com/elm/url/pull/35

AlexanderEkdahl avatar Oct 05 '19 21:10 AlexanderEkdahl

we are having this issue as well

venkatbvc avatar Jul 24 '20 13:07 venkatbvc

@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.

venkatbvc avatar Apr 14 '21 14:04 venkatbvc

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

sriharshabm avatar Jun 04 '21 06:06 sriharshabm

Could you please update on this ticket

sriharshabm avatar Aug 02 '21 06:08 sriharshabm

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

Could you pls comment on this

sriharshabm avatar Jan 31 '22 17:01 sriharshabm

@evancz @MrL1605 Could you please help on this.

SarthakSahu avatar Feb 07 '22 15:02 SarthakSahu