Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: replace ftp tls impl as rustls #3760

Merged
merged 3 commits into from
Dec 14, 2023

Conversation

oowl
Copy link
Member

@oowl oowl commented Dec 14, 2023

close #3625

@oowl
Copy link
Member Author

oowl commented Dec 14, 2023

After the current change.

╰─$  cargo tree -i openssl --all-features --target all                                                                                                                                                                                             130 ↵
openssl v0.10.59
└── native-tls v0.2.11
    ├── hyper-tls v0.5.0
    │   └── reqwest v0.11.22
    │       ├── opendal v0.43.0 (/home/ouyang/code/incubator-opendal/core)
    │       └── reqsign v0.14.4
    │           └── opendal v0.43.0 (/home/ouyang/code/incubator-opendal/core)
    ├── redis v0.23.3
    │   └── opendal v0.43.0 (/home/ouyang/code/incubator-opendal/core)
    ├── reqwest v0.11.22 (*)
    └── tokio-native-tls v0.3.1
        ├── hyper-tls v0.5.0 (*)
        ├── redis v0.23.3 (*)
        └── reqwest v0.11.22 (*)

core/Cargo.toml Outdated Show resolved Hide resolved
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Xuanwo Xuanwo merged commit 3741397 into main Dec 14, 2023
208 checks passed
@Xuanwo Xuanwo deleted the dep/superbase-replace-by-native-tls branch December 14, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

suppaftp enforce to use native-tls
2 participants