oscerd opened a new pull request, #2961: URL: https://github.com/apache/camel-kamelets/pull/2961
Fixes #2953 `sqlserver-sink` and `sqlserver-source` declared: ```yaml encrypt: default: false trustServerCertificate: default: true ``` and interpolated both into the JDBC URL. The Microsoft SQL Server JDBC driver has defaulted `encrypt` to `true` since version 10.2, so the Kamelets were overriding the driver's own default in the weaker direction, and pairing it with `trustServerCertificate=true` (which suppresses certificate validation). This PR flips both to the driver-aligned values and clarifies the `trustServerCertificate` description to say what it actually does. ### Behaviour change This changes the default for existing deployments. A deployment pointing at a SQL Server without a certificate the client trusts will now need `encrypt=false` or `trustServerCertificate=true` set explicitly. `camel-kamelets` has no upgrade guide of its own, so the note belongs in the `apache/camel` upgrade guide for the release that picks this up — flagging for a maintainer to place it. ### Verification - `script/validator` reports no errors - `mvn verify` passes --- _Claude Code on behalf of Andrea Cosentino_ -- 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]
