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

Fix SSHConnectionPool with subdirectories in URI #3273

Merged
merged 1 commit into from
Apr 24, 2022

Conversation

TranceLove
Copy link
Collaborator

@TranceLove TranceLove commented Apr 23, 2022

Description

This is to allow sharing connections having the same connection credentials and host, but with different paths. Also fixes problem that thumbnails on SSH servers cannot be generated due to failure on loading remote file.

Automatic tests

  • Added test cases

Manual tests

  • Done

  • Device: Pixel 2 emulator

  • OS: Android 11
    Image thumbnails should load with SSH connections setup with default path provided.

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

This is to allow sharing connections having the same connection credentials and host, but with different paths. Also fixes problem that thumbnails on SSH servers cannot be generated.
@TranceLove TranceLove added Issue-Bug Related unexpected behavior or something worth investigating. Area-Ssh/Scp/Sftp labels Apr 23, 2022
@VishalNehra VishalNehra merged commit 702a63f into release/3.7 Apr 24, 2022
@VishalNehra VishalNehra added this to the v3.7 milestone Apr 24, 2022
@TranceLove TranceLove deleted the bugfix/ssh-connection-pool-fix-subpath branch April 26, 2022 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Ssh/Scp/Sftp Issue-Bug Related unexpected behavior or something worth investigating.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants