Hello, I would like to contribute to the development of this feature. Could you tell me which part of the code I should read , or provide some other relevant information? @Xuanwo

Hi, @ozewr, thank you for your interest!

Memcached currently interacts directly with TcpStream:

We need to add support for using a TLS library like tokio_rustls to establish a TLS connection. For example: https://github.com/rustls/tokio-rustls/blob/main/examples/client.rs

Hi, @ozewr, thank you for your interest!

Memcached currently interacts directly with TcpStream:

opendal/core/src/services/memcached/binary.rs

Lines 104 to 109 in f0ace56

	impl Connection {
	pub fn new(io: TcpStream) -> Self {
	Self {
	io: BufReader::new(io),
	}
	}

We need to add support for using a TLS library like tokio_rustls to establish a TLS connection. For example: https://github.com/rustls/tokio-rustls/blob/main/examples/client.rs

Hello, I'm a new programmer, so I want to share my thoughts and see if they are correct. It seems that here we need a new struct implementing bb8::ManageConnection, similar to MemcacheConnectionManager. However,

impl bb8::ManageConnection for MemcacheConnectionManager {
    type Connection = TlsConnection { io: BufReader<TlsStream<TcpStream>> };
    type Error = Error;
...

Or directly modify the implementation of bb8::ManageConnection for MemcacheConnectionManager to make it only support TLS connections. @Xuanwo

Maybe we can make binary::Connection an enum that have TcpConnection and TlsConnection.

@Xuanwo I wanna take up this task.

@Xuanwo I wanna take up this task.

Hi, @ozewr is working on this now. Maybe we can collaborate with them first?

@Xuanwo I wanna take up this task.

Hi, @ozewr is working on this now. Maybe we can collaborate with them first?

Sorry, I’ve been a bit busy with my day job this week, but I think I should be able to submit a PR tomorrow. I think we can collaborate on it. @geetanshjuneja