gnodet-bot commented on code in PR #28186: URL: https://github.com/apache/camel/pull/28186#discussion_r4326229044
########## docs/user-manual/modules/ROOT/pages/camel-4x-upgrade-guide-4_24.adoc: ########## @@ -1405,6 +1405,17 @@ MINA does not pass a failed handshake on as an error: the producer fails with a `org.apache.mina.filter.ssl.SslHandler`. A producer that starts timing out after the upgrade should be checked for a certificate that does not list the host. +=== camel-ftp - the host name of the FTPS server is verified + +The `ftps` endpoint now checks that the certificate of the server matches the host of the endpoint, with the same +rules as an HTTPS client, with `sslContextParameters` as well as with the `ftpClient.trustStore.*` options. Before, any certificate accepted by the trust manager was accepted, whatever host the Review Comment: 📝 **Nit:** Line 1411 is ~193 characters. The adjacent camel-mina section and the rest of the file wrap at ~120. Split after `options.` to stay consistent: ```suggestion The `ftps` endpoint now checks that the certificate of the server matches the host of the endpoint, with the same rules as an HTTPS client, with `sslContextParameters` as well as with the `ftpClient.trustStore.*` options. Before, any certificate accepted by the trust manager was accepted, whatever host the certificate was issued for. ``` -- 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]
