Nephery commented on PR #13041:
URL: https://github.com/apache/camel/pull/13041#issuecomment-1932801270

   > In my opinion it doesn't make sense to have both the parameters as path 
and query. I would leave just the url as path and add the token as mandatory 
query parameter. The proposed solution will confuse end user and since we are 
going to release 4.4.0 soon, I think we could remove the token as path 
parameter, add it only as mandatory query parameter and document it in the 
migration guide.
   
   I adjusted the PR according to @oscerd's proposal.


-- 
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-unsubscr...@camel.apache.org

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

Reply via email to