Consider making HttpUrl.getChecked() public

[grodin]

Using HttpUrl.parse() in Kotlin to parse constant urls that you know will parse is slightly annoying because of the nullable return.

I've worked around it by writing a static method parseHttpUrlOrThrow() but this essentially duplicates the existing HttpUrl.getChecked(), which is unfortunately package private. It would be useful if this method was public.

[JakeWharton]

We've talked about this for a long time. It's caused at least one bug in Retrofit and OkHttp thus far.

[swankjesse]

Isn't it just parse(url)!! ?

[JakeWharton]

Assuming you remember that, yes. In Java you don't. And in Kotlin it's a platform type so you sometimes are forced to.

[JakeWharton]

Another advantage of a throwing version is that it's able to communicate useful information about the reason why it failed.

[shaburov]

+1

[yschimke]

oops PR hasn't landed yet