Nabil Abdel-Hafeez comments

Results 242 comments of


                                            Nabil Abdel-Hafeez

Fix Route merge order to fix Issue #3066

@jdegoes I find this behaviour confusing. Everywhere else we give "the new thing" precedence. Should we change it?

Fix Route merge order to fix Issue #3066

closing in favor of #3337

OpenAPI spec to endpoint generator does not recognize field validation

Your described solution should be the right way. However, I am not sure that all Open API validations can be mapped to current schema validations. It might need some extension...

OpenAPI spec to endpoint generator does not recognize field validation

/livestream [**Click here to watch 987Nabil**](https://tv.algora.io/gh/7283535/channel)

OpenAPI spec to endpoint generator does not recognize field validation

I am done with the impl. But another fix for zio-schema is needed https://github.com/zio/zio-schema/pull/698

Malformed request body failed to decode: Unexpected end of input

Please provide code for a reproducer

Trailing path routes override all Method.ANY routes (breaks Tapir interop)

@godspeedelbow I am not sure we can solve this in zio-http. A `GET` should always take precedence over an `ANY`. That is works as designed. Only if the path does...

Trailing path routes override all Method.ANY routes (breaks Tapir interop)

ANY is explicitly designed to be a catch all with the lowest precedence. Only your use case would work better if the behavior would be different. /proxy could be a...

Trailing path routes override all Method.ANY routes (breaks Tapir interop)

I kinda dislike adding complexity for everyone to help out tapir. What is tapir doing internally? I think that might be where a fix is applied easier.

Add access to Env in error handler

@jdegoes I tried this and came to have an issue with this definition of Route ```scala private final case class Augmented[InEnv, -OutEnv, +Err]( route: Route[InEnv, Err], aspect: Handler[InEnv, Response, Request,...