```println Url <https www google com unicorns> pathSegments kotlinlang #ktor

Channels

100daysofcode

100daysofkotlin

100daysofkotlin-2021

androidgithubprojects

cincinnati-user-group

competitive-programming

event21-community-content

java-to-kotlin-refactoring

jvm-ir-backend-feedback

kotlin-multiplatform-contest

kotlinultimatechallenge

myndocs-oauth2-server

naming

navigation-architecture-component

practical-functional-programming

proguard

prozis-android-backup

refactoring-to-kotlin

russian-kotlinasfirst

skia-wasm-interop-temp

stayhungrystayfoolish

swagger-gradle-codegen

tricity-kotlin-user-group

Title

Paul Woitaschek

05/11/2022, 10:06 AM

println(Url("<https://www.google.com/unicorns>").pathSegments)

Is it a bug that this returns a list of two strings, where the first is empty and the second one is unicorns? Imo this should only contain

unicorns

@e5l Do I understand correctly, that this is a bug in 2.0.1 and not in 2.0.0 that’s now fixed by https://github.com/ktorio/ktor/pull/2975 in main?

➕ 1

Aleksei Tirman [JB]

05/11/2022, 10:26 AM

According to the test this behavior is expected.

Grégory Lureau

05/11/2022, 10:37 AM

What means line 330 in this test then? Looks a bit weird... Also l.361 matches this specific case.

Aleksei Tirman [JB]

05/11/2022, 10:40 AM

I would say line 365

👍 1

I agree that empty path segments for trailing slashes looks strange

Paul Woitaschek

05/11/2022, 10:48 AM

It's also unexpected and has caused a bug here (which was thankfully covered by a test 😅)

Issue opened: https://youtrack.jetbrains.com/issue/KTOR-4402/pathSegments-adds-empty-string

For some reason it’s tagged as 2.0.1 and I can’t change that

Aleksei Tirman [JB]

05/30/2022, 6:02 PM

I’ve added an explanation for this behavior in a comment.

Paul Woitaschek

05/30/2022, 6:31 PM

Thanks. Added more explanation why I think this is a bad idea

2 Views

#ktor Join Slack