Document that an exception thrown in a Path param unmarshaller is a rejection about akka-http HOT 9 CLOSED

Good to see you here @ilke-zilci!

from akka-http.

Hey, I'd like to take this one, is anyone working on it already?

from akka-http.

No, seems no one has picked it up yet, thanks for looking into it! :)
(Feel free to ask if not sure about things with it etc)

from akka-http.

I agree, might be a good starting contribution!

However, I find the ticket description a bit unclear. I think it's about using parameter("age".as(xyzUnmarshaller)) and this xyzUnmarshaller throws an exception? Do you remember if that was the case here, @ktoso?

In that case, this code here seems to convert the exception into a rejection: https://github.com/akka/akka-http/blob/master/akka-http/src/main/scala/akka/http/scaladsl/server/directives/ParameterDirectives.scala#L115

It should probably be documented in the documentation of the parameters directive here: https://github.com/akka/akka-http/blob/master/docs/src/main/paradox/scala/http/routing-dsl/directives/parameter-directives/parameters.md

from akka-http.

Yes, it was about that; AFAIR someone expected "well it's an exception so should be an error page!", but indeed there we interpret it as "well, unable to extract, let's try another one", so as a rejection :)

from akka-http.

Great! I will work on it then. It goes here, right? https://github.com/akka/akka-http/blob/master/docs/src/main/paradox/scala/http/routing-dsl/rejections.md

from akka-http.

As @jrudolph mentioned, I would document it next to the Path Parameter documentation pages (https://github.com/akka/akka-http/blob/master/docs/src/main/paradox/scala/http/routing-dsl/directives/parameter-directives/parameters.md) and then link to the rejection doc page as a "see for further info".

And don't forget to document it for both Scala and Java docs :)

from akka-http.

Ok thanks. And sorry missed that line on the first read.

from akka-http.

Fixed in #1285.

from akka-http.