gnodet opened a new pull request, #23332:
URL: https://github.com/apache/camel/pull/23332
## Summary
Improve `PomSanitizer` in `camel-jbang-mcp` to close detection gaps and
simplify internals:
- **CDATA support**: values wrapped in `<![CDATA[...]]>` were previously
invisible to detection — now inspected and masked
- **Camel placeholder recognition**: `{{vault:...}}` and `{{...}}`
placeholders are preserved alongside Maven `${...}`
- **URL-embedded credentials**: new pattern detects `://user:password@host`
in any element (catches JDBC URLs, HTTP basic auth in non-sensitive-named
elements like `<db.url>`)
- **Additional keywords**: `connection-string`, `connectionstring`,
`connection_string`
- **Single-pass processing**: merged the separate detection and masking
regex passes into one
- **Minor cleanup**: `String.join` replaces `Collectors.joining`, removed
unused import
12 new test cases covering all additions.
## Test plan
- [x] `mvn test -Dtest=PomSanitizerTest` — all 30 tests pass
- [ ] Verify no regressions in existing MCP tool behavior
_Claude Code on behalf of Guillaume Nodet_
--
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]
