-
Notifications
You must be signed in to change notification settings - Fork 594
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
http: add handlePF directive #3367
Conversation
Test FAILed. !!! Couldn't read commit file !!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a good direction to me. Nice to also have the Sync
variants. Likely they'll be less common so indeed good to give those the more verbose name.
akka-http/src/main/scala/akka/http/scaladsl/server/directives/RouteDirectives.scala
Outdated
Show resolved
Hide resolved
akka-http/src/main/scala/akka/http/scaladsl/server/directives/RouteDirectives.scala
Outdated
Show resolved
Hide resolved
Test FAILed. Pull request validation reportFailed Test SuitesTest result for 'akka-http-tests / Pr-validation / ./ executeTests'
|
and merge documentation pages
Test FAILed. Pull request validation reportFailed Test SuitesTest result for 'akka-http-tests / Pr-validation / ./ executeTests'
Test result for 'docs / Pr-validation / ./ executeTests'
|
Test FAILed. Pull request validation reportFailed Test SuitesTest result for 'docs / Pr-validation / ./ executeTests'
|
Test PASSed. |
Nothing to add here from my side. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Might be interesting to think about if we could come up with something for the javadsl, but it's not so obvious what a PartialFunction should look like there.
Yes, good point, I think akka-grpc also generates the scala signatures for |
Not currently, but if we add a directive to convert a scala (it currently returns a non-partial java function with a 'magic constant' representing 'no return value') |
Refs #3239, #3263
Before I continue, I'd like some feedback on API and naming.