You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/11/27 10:14:44 UTC
[GitHub] [camel-quarkus] jamesnetherton opened a new issue #2042: Configure order for routes added by platform-http consumer
jamesnetherton opened a new issue #2042:
URL: https://github.com/apache/camel-quarkus/issues/2042
This [conversation](https://camel.zulipchat.com/#narrow/stream/257302-camel-quarkus/topic/Writing.20tests) lead to an interesting discovery about how the Vert.x router route order can affect the behaviour of the platform-http consumer.
TL;DR the user set up a REST route which expected to match `method=POST` & `Content-Type=application/json`. When the correct content type was not provided by the client, a 404 response was returned instead of the expected 415.
Turns out that because `META-INF/resources` was present in the source tree, Quarkus registers Vert.x route(s) to handle the static resources. These are registered [after](https://github.com/quarkusio/quarkus/blob/4db67556839a7ef7072515539d226b872e2467c6/extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/VertxHttpRecorder.java#L111) the Camel ones. Thus the 415 response is silently dropped because the static route handlers can't match anything and return a 404.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] jamesnetherton commented on issue #2042: Configure order for routes added by platform-http consumer
Posted by GitBox <gi...@apache.org>.
jamesnetherton commented on issue #2042:
URL: https://github.com/apache/camel-quarkus/issues/2042#issuecomment-848986037
I did start working on a solution. But after this [conversation](https://github.com/quarkusio/quarkus/issues/14112) I abandoned it.
If it becomes a frequent issue for folks, then maybe we invest more time on it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] ppalaga commented on issue #2042: Configure order for routes added by platform-http consumer
Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #2042:
URL: https://github.com/apache/camel-quarkus/issues/2042#issuecomment-848973762
Do we need to do something about this?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org