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 2022/03/05 16:04:49 UTC

[GitHub] [camel] vagpap opened a new pull request #7118: CAMEL-17751: Extracting Camel-Saga-Compensate & Camel-Saga-Complete f…

vagpap opened a new pull request #7118:
URL: https://github.com/apache/camel/pull/7118


   If the verifyRequest method is not able to retrieve:
   
   `Camel-Saga-Compensate` & `Camel-Saga-Complete`
   
   as individual headers, try to extract them from the `CamelHttpQuery` header.
   If found, add them also as headers to the exchange.getIn since if they are not there, neither the compensation or the completion is called correctly.
   


-- 
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] vagpap commented on pull request #7118: CAMEL-17751: Extracting Camel-Saga-Compensate & Camel-Saga-Complete f…

Posted by GitBox <gi...@apache.org>.

vagpap commented on pull request #7118:
URL: https://github.com/apache/camel/pull/7118#issuecomment-1059982493


   The code currently throws a RuntimeCamelException in case the URISupport.parseQuery method throws the URISyntaxException.
   
   Is this appropriate? Should it throw something else?


-- 
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 #7118: CAMEL-17751: Extracting Camel-Saga-Compensate & Camel-Saga-Complete f…

Posted by GitBox <gi...@apache.org>.

davsclaus commented on pull request #7118:
URL: https://github.com/apache/camel/pull/7118#issuecomment-1060012292


   I think we can cherry pick to main so lets try this first


-- 
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 #7118: CAMEL-17751: Extracting Camel-Saga-Compensate & Camel-Saga-Complete f…

Posted by GitBox <gi...@apache.org>.

davsclaus merged pull request #7118:
URL: https://github.com/apache/camel/pull/7118


   


-- 
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] vagpap commented on pull request #7118: CAMEL-17751: Extracting Camel-Saga-Compensate & Camel-Saga-Complete f…

Posted by GitBox <gi...@apache.org>.

vagpap commented on pull request #7118:
URL: https://github.com/apache/camel/pull/7118#issuecomment-1060011929


   @oscerd should I do it and then open a new PR for main? 


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