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

SftpConnectDialog: Wrap open DocumentsUI intent with Intent.createChooser() to get PEM content #4234

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

TranceLove
Copy link
Collaborator

@TranceLove TranceLove commented Aug 5, 2024

Description

To fix regressions in newer Androids that SftpConnectDialog unable to load PEM for SSH authentication.

Reference: https://stackoverflow.com/questions/10002065/android-gallery-always-returns-result-canceled-to-onactivityresult

Manual tests

  • Done

  • Device: Fairphone 3

  • OS: LibremobileOS 5.0 (Android 14)

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

@TranceLove TranceLove force-pushed the bugfix/sftpconnectdialog-select-pem branch from 329f8c2 to da70e2e Compare August 5, 2024 07:19
@TranceLove TranceLove changed the base branch from release/4.0 to release/3.10 August 5, 2024 07:19
@TranceLove TranceLove added the Issue-Bug Related unexpected behavior or something worth investigating. label Aug 5, 2024
@TranceLove TranceLove force-pushed the bugfix/sftpconnectdialog-select-pem branch 2 times, most recently from 5643117 to 8d17631 Compare August 5, 2024 15:26
@TranceLove TranceLove force-pushed the bugfix/sftpconnectdialog-select-pem branch 2 times, most recently from 782fb35 to d1f7bd1 Compare August 5, 2024 15:45
…oser() to get PEM content

To fix regressions in newer Androids that SftpConnectDialog unable to load PEM for SSH authentication.

Reference: https://stackoverflow.com/questions/10002065/android-gallery-always-returns-result-canceled-to-onactivityresult
@TranceLove TranceLove force-pushed the bugfix/sftpconnectdialog-select-pem branch from d1f7bd1 to 2410b08 Compare August 5, 2024 15:51
@VishalNehra VishalNehra merged commit f4eec02 into release/3.10 Aug 20, 2024
3 checks passed
@VishalNehra VishalNehra added this to the v4.0 milestone Aug 20, 2024
@VishnuSanal VishnuSanal deleted the bugfix/sftpconnectdialog-select-pem branch August 20, 2024 14:21
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.

2 participants