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 2021/11/03 07:18:12 UTC

[GitHub] [camel] henka-rl opened a new pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

henka-rl opened a new pull request #6372:
URL: https://github.com/apache/camel/pull/6372


   Added tests for this in jetty, servlet and undertow components
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus merged pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus merged pull request #6372:
URL: https://github.com/apache/camel/pull/6372


   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958706918






-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958712537


   Can you please fix the code style? Thanks. 


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958712537


   Can you please fix the code style? Thanks. 


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958706918






-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-960920042


   @henka-rl I created a JIRA
   https://issues.apache.org/jira/browse/CAMEL-17166
   
   You are welcome to send PRs


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-959090120


   Can you check what camel-jetty does?


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-959059945


   > > I also found a strange behaviour in undertow. Then the body is null undertow it sets the body to "No response available":
   > > https://github.com/apache/camel/blob/b87886bfe576e4ccb5513d9bb98a45a903711464/components/camel-undertow/src/main/java/org/apache/camel/component/undertow/UndertowConsumer.java#L249
   > 
   > Can you try git blame to find out if there is a JIRA or more? Also check if other http component does that.
   
   I found CAMEL-14025


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958706918


   I also found a strange behaviour in undertow. Then the body is null undertow it sets the body to "No response available":
   https://github.com/apache/camel/blob/b87886bfe576e4ccb5513d9bb98a45a903711464/components/camel-undertow/src/main/java/org/apache/camel/component/undertow/UndertowConsumer.java#L249


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958706918






-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958731176






-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958712537


   Can you please fix the code style? Thanks. 


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958716230


   In progress...
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus merged pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus merged pull request #6372:
URL: https://github.com/apache/camel/pull/6372


   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-960479274


   Thanks can you create a JIRA. Then lets fix this to make them do the same, to return an empty body.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus merged pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus merged pull request #6372:
URL: https://github.com/apache/camel/pull/6372






-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] henka-rl commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
henka-rl commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-960289901


   camel-jetty returns an empty ("") response:
   ![image](https://user-images.githubusercontent.com/67908385/140232271-396eb536-e1f6-49df-ad50-21b0573f7355.png)
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958731176


   > I also found a strange behaviour in undertow. Then the body is null undertow it sets the body to "No response available":
   > 
   > https://github.com/apache/camel/blob/b87886bfe576e4ccb5513d9bb98a45a903711464/components/camel-undertow/src/main/java/org/apache/camel/component/undertow/UndertowConsumer.java#L249
   
   Can you try git blame to find out if there is a JIRA or more? Also check if other http component does that.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958712537


   Can you please fix the code style? Thanks. 


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6372: [CAMEL-17159] Changes to support multiple valid content types

Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6372:
URL: https://github.com/apache/camel/pull/6372#issuecomment-958731176






-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org