ramu11 commented on PR #29592: URL: https://github.com/apache/camel/pull/29592#issuecomment-6796517491
Thanks for the review and for reproducing the paused-request issue. I've addressed both remaining findings: 1. **Paused request on OAuth rejection:** `checkOAuth` now resumes the request before ending the response in all three rejection paths: validator infrastructure failure (503), unauthenticated requests (400/401), and insufficient scope (403). 2. **Regression coverage:** Added a regression test that sends a 300 KB POST and then a follow-up GET over the same TCP connection. It covers all three rejection cases and verifies that the connection remains usable. 3. **Generated catalog documentation:** Regenerated and committed the catalog copy of `mcp-server.adoc`, including the corrected camel-oauth link and RFC 9728 origin-root explanation. Pushed in commit: `be65fae026eb` — `CAMEL-26356: Resume rejected MCP OAuth requests` Thanks again for the detailed review and the reproducer. Please take another look. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
