yihong0618 commented on code in PR #5716:
URL: https://github.com/apache/opendal/pull/5716#discussion_r1987359244
##########
core/src/services/ftp/backend.rs:
##########
@@ -221,13 +223,21 @@ impl bb8::ManageConnection for Manager {
async fn connect(&self) -> Result<Self::Connection, Self::Error> {
let stream = ImplAsyncFtpStream::connect(&self.endpoint).await?;
- // switch to secure mode if ssl/tls is on.
let mut ftp_stream = if self.enable_secure {
+ // Create a root certificate store
+ let root_store = RootCertStore::empty();
+ // Optionally, add certificates (e.g., system certs or custom ones)
+ // For now, we'll leave it empty as an example
Review Comment:
seems no need follow the old `default` will change the comment
--
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]
