davsclaus opened a new pull request, #24927:
URL: https://github.com/apache/camel/pull/24927

   _Claude Code on behalf of davsclaus_
   
   ## Summary
   
   Fix three parsing bugs in `CxfRsProducer` that throw on legal RFC 3986 URIs 
in consumer→producer bridge scenarios:
   
   1. **`setupClientMatrix`** — `StringIndexOutOfBoundsException` when `;` 
appears after `?` in the request URI (e.g., `/path?foo=bar;semi`). Fixed by 
computing `matrixStart` only within the path portion (before `?`).
   
   2. **`getMatrixParametersFromMatrixString`** — `IllegalArgumentException` on 
valueless matrix params (e.g., `/path;flag`). Fixed by treating valueless 
segments as empty-value params instead of throwing.
   
   3. **`insertQueryParametersFromQueryString`** — `IllegalArgumentException` 
on valueless query params (e.g., `?debug&key=value`). Fixed by treating 
valueless tokens as empty-value params instead of throwing.
   
   All three bugs allow an external client to force an HTTP 500 with a 
well-formed request.
   
   ## Test plan
   
   - [x] Added `testValuelessQueryParameter` — sends `flag&key=value` query 
string
   - [x] Added `testQueryParameterWithEmptyValue` — sends `name=&key=value` 
query string  
   - [x] Added `testMultipleValuelessQueryParameters` — sends 
`debug&trace&verbose` query string
   - [x] Existing `CxfRsProducerHttpMethodHeaderTest` passes (regression check)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>


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

Reply via email to