silver-ymz opened a new pull request, #2186:
URL: https://github.com/apache/incubator-opendal/pull/2186

   Impl #708
   
   This PR adds support for the sftp backend using the 
[openssh-sftp-client](https://github.com/openssh-rust/openssh-sftp-client) 
library.
   
   ## Misc
   In the `Connection` struct, the sftp connection is stored while hiding the 
`session` and `channel` used to create the connection. However, the `Channel` 
definition requires the 'sess lifetime in its type annotation, while `Session` 
does not. To address this, the `OwningHandle` is used to ensure that `Channel` 
has ownership of `Session` to pass the lifetime check. A similar situation 
exists for `SftpReader`.
   
   ## Tests
   All tests have passed except for `test_list_rich_dir`, which seems to be an 
upstream issue. The problem is that it misses some files when listing a folder 
with many files, and an issue has been opened for it at 
https://github.com/openssh-rust/openssh-sftp-client/issues/62.


-- 
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