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

[Feature Request] Support for Client Certificates (mTLS) #126

Open
3 tasks done
iDontWantAUsername opened this issue Sep 10, 2024 · 2 comments
Open
3 tasks done

[Feature Request] Support for Client Certificates (mTLS) #126

iDontWantAUsername opened this issue Sep 10, 2024 · 2 comments

Comments

@iDontWantAUsername
Copy link

Checklist

  • I have checked and there are no existing issues - open or closed - which request the same feature.
  • I have taken the time to fill in all the required details. I understand that the feature request will be dismissed otherwise.
  • This issue contains only one feature request.

Feature description

Now that the official NextCloud Android app supports Client Certificates it would be great if this app did too. This would mean when authenticating for the first time and a client certificate is required a system prompt would be show to select which already installed certificate to use.
The certificate used will be remembered by the app and used for all future connections to the same URL.

Why do you want this feature?

Client Certificates are now supported by the NextCloud official Android app allowing an additional layer of security. This means that in order to access a NextCloud instance an additional certificate may be required that is installed on the device. This would allow access to passwords outside of my local network without a VPN via the app in the same way that the NextCloud app now works.

Additional information

NextCloud Android PR: nextcloud/android-library#1308

@hegocre
Copy link
Owner

hegocre commented Sep 12, 2024

Hello @iDontWantAUsername

I've taken a look at the code needed to do it, and even though it does not look too difficult, I have no environment to test it, so I can't implement it right now. If I have time in the future and I am less busy, I will try to setup a test instance with mTLS to check, but right now I'm unable.

Thank you!

@iDontWantAUsername
Copy link
Author

All good, thanks for the update, and your work on the project so far!

It is a nice to have future feature so not essential. Happy to help with some beta testing going forward if required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants