nextcloud / news-android

📱🗞️ Android client for the Nextcloud news/feed reader app
https://play.google.com/store/apps/details?id=de.luhmer.owncloudnewsreader
GNU Affero General Public License v3.0
697 stars 258 forks source link

Feature request: Support for mTLS / client certificates #1430

Closed DeLoooping closed 5 months ago

DeLoooping commented 5 months ago

Effectively the title.

As the Android NextCloud app supports the usage of client certificates now (since v3.29.0), it would be great if the news app supported that too.

stefan-niedermann commented 5 months ago

News Android uses the Nextcloud Single Sign On library. It therefore does not make any HTTP calls itself but proxies all requests through the Nextcloud Android app. I therefore expected it to support this newly added feature implicitly. What makes you think that this is not the case? Can you post any error messages or screenshots?

DeLoooping commented 5 months ago

Yes, you are right. When logging in via the Nextcloud Files App, it works.

When logging in manually, you get a 400: nextcloud_news_400

stefan-niedermann commented 5 months ago

Manual login is deprecated (and this issue is one of the reasons :slightly_smiling_face:) I doubt that anyone will invest any further time into the manual login to be honest, but let's wait for a statement of the maintainer @David-Development

David-Development commented 5 months ago

@stefan-niedermann You've said it all 😁 Manual login is deprecated and in case someone experiences issues with it it's time to switch to Single Sign On