oscerd opened a new issue, #2953:
URL: https://github.com/apache/camel-kamelets/issues/2953

   `sqlserver-sink` and `sqlserver-source` declare these defaults:
   
   ```yaml
   encrypt:
     type: boolean
     default: false
   trustServerCertificate:
     type: boolean
     default: true
   ```
   
   and interpolate them into the JDBC URL:
   
   ```
   
jdbc:sqlserver://{{serverName}}:{{serverPort}};databaseName={{databaseName}};encrypt={{encrypt}};trustServerCertificate={{trustServerCertificate}};
   ```
   
   The Microsoft SQL Server JDBC driver has defaulted `encrypt` to `true` since 
version 10.2, so the Kamelets actively override the driver's own default in the 
weaker direction — and pair it with `trustServerCertificate=true`, which 
suppresses certificate validation.
   
   Proposal: flip both to the driver-aligned values (`encrypt: true`, 
`trustServerCertificate: false`), and note the change in the upgrade guide 
since it affects existing deployments against servers without a trusted 
certificate.
   
   Affected:
   - `kamelets/sqlserver-sink.kamelet.yaml`
   - `kamelets/sqlserver-source.kamelet.yaml`
   
   _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]

Reply via email to