[ https://issues.apache.org/jira/browse/CAMEL-11162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16494914#comment-16494914 ]
ASF GitHub Bot commented on CAMEL-11162: ---------------------------------------- GitHub user davsclaus opened a pull request: https://github.com/apache/camel/pull/2354 CAMEL-11162: rest-dsl add client request validation So we check whether required data is included in the client http request when being processed by rest-dsl consumer. We now check for missing - query parameters - http headers - body The context-path is already handled in the uri path matcher, so it will fail already for invalid context-path. This can be turned on in the rest configuration. You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/camel rest-val Alternatively you can review and apply these changes as the patch at: https://github.com/apache/camel/pull/2354.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2354 ---- commit 8f088e7fa1fb11e48ed4150ad460bad1d4dd1020 Author: Claus Ibsen <claus.ibsen@...> Date: 2018-05-29T14:38:28Z CAMEL-11162: rest-dsl now has validation check for consumer to check if the http client request has content-type/accept headers that is supported in rest-dsl consumes/produces settings. commit c9ff0ce02d9d7ed72e84b5fb58a19d7820822300 Author: Claus Ibsen <claus.ibsen@...> Date: 2018-05-30T06:45:56Z CAMEL-11162: rest-dsl now has validation check for consumer to check if the http client request has content-type/accept headers that is supported in rest-dsl consumes/produces settings. commit 066566dea22ed6dcc2978ca33b0707a6d0035f7f Author: Claus Ibsen <claus.ibsen@...> Date: 2018-05-30T09:03:49Z CAMEL-11162: rest-dsl now has validation check for consumer to check if the http client request has content-type/accept headers that is supported in rest-dsl consumes/produces settings. ---- > camel-rest - Should we add content-type check for server side > ------------------------------------------------------------- > > Key: CAMEL-11162 > URL: https://issues.apache.org/jira/browse/CAMEL-11162 > Project: Camel > Issue Type: Improvement > Components: camel-core > Reporter: Claus Ibsen > Assignee: Claus Ibsen > Priority: Major > Fix For: 2.22.0 > > > For example setting up a rest-dsl which consumes application/json and then a > client calls it with text/plain or application/xml, should we then automatic > let rest consumer detect this and return a HTTP status 415 (unsuported media > type) > Not all the HTTP server components does this today, eg jetty etc. But when > using restlet which is more natual REST it would do so. > We could then add an option to turn this on|off. > The check is only if the media-type is within the list that may have been > specified on consumes in the rest-dsl. -- This message was sent by Atlassian JIRA (v7.6.3#76005)