How do I make this locations match? The first shou...

# ktor

[degreat]

How do I make this locations match? The first should match alphanumeric, and the second should match integers

[louiscad]

Is there a way to define priority?

[Matteo Mirk]

I’m just guessing here: probably to Ktor a String path variable will match everything, so it stops. Maybe declaring the {id} first will make it try to parse an integer and pass to the next if it fails… but yeah it’s not a robust way.

[degreat]

I tired that, but didn't work

[degreat]

It didn't work for both cases

[degreat]

If I switch positions, the second match fails

[Matteo Mirk]

perhaps this is the feature that could help you: https://ktor.io/docs/routing.html#extensibility

[Matteo Mirk]

sorry I missed a sentence:

Path parsing is not extensible.

😞

[degreat]

Alright thanks, I'll try that and let you know the outcome

[degreat]

Thanks very much

[Matteo Mirk]

I’m starting to suspect that the only way to solve this is to match a generic {id} path and inside the routing function have a factory create either SlugOne or IdOne depending on a regex. Maybe it’s solvable only by code…

[Matteo Mirk]

you’re welcome, let us know if you make it!

[degreat]

@Matteo Mirk have a look at the trace

[degreat]

It says lost in ambiguity tie

[degreat]

And then there's stack trace that, it cannot convert the string into an integer

[degreat]

So an approach by the ktor team, imo, should be instead of throwing away ambiguity ties, it should be kept just in case a first match raises a parse/deserialization error

[degreat]

Until it's resolved, this is what I settled with. Probably this weekend I may try to solve the Locations approach and submit an MR

[Matteo Mirk]

Cool, that’s probably the only way to solve it for now. My knowledge stops a the trace, I’d have to dig into the path matching code to understand what’s going on. We should really need an authoritative voice from the Ktor team, but no one has answered so far… Maybe open an issue on Ktor repo and see what they say. Thanks for updating me!

[degreat]

I already created the issue, but there is no response yet.

[Matteo Mirk]

Don’t lose hope, maybe they’ll get to it. Can you share the issue link, I could vote on it, if it helps

[degreat]

Haha, thanks.

[degreat]

https://youtrack.jetbrains.com/issue/KTOR-1050

[Matteo Mirk]

if it’s of any help, I found the code where the route resolution is performed: https://github.com/ktorio/ktor/blob/7e524156e0357763595dde457d8c922627b8def7/ktor-server/ktor-server-core/jvm/src/io/ktor/routing/RoutingResolve.kt#L102

[degreat]

this is great. Really appreciate the effort. I'll try and provide a fix and submit a merge request.. and hope they merge it in