Question: Drop use of androids build in AccountManager
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- java
- Domain
- authentication, mobile
Research direction
Start with the referenced AccountImporter.java in the Android-SingleSignOn library and review how AccountManager currently supplies accounts. Then inspect the proposed files-app integration and custom activity flow. Done means the files app can list available accounts and return the user's selection without requiring the client to use AccountManager.
Written by the indexing model from the issue text.
Description
@tobiasKaminsky Just throwing in some thoughts here. As we had a bunch of issues already when trying to figure out which accounts exist, I was thinking about if we can drop the need to use the Android "AccountManager" at all. As of Android 9 getting accounts from another app is even more difficult. Therefore dropping it makes things easier for all involved parties.
Advantages:
- reauthentication works without the need to request the account permission again (way easier for client apps as we don't need to restart the login flow)
- Custom dialog for choosing the account (nextcloud design / theme)
- reduce complexity of account handling (in the sso library)
Disadvantages:
- None?
So basically what I'm proposing is:
- Instead of calling the account manager to show us available accounts, we create a new custom activity in the files app for that. (Reference)
- The files app lists all accounts available (as the files app has the required permission for that)
- The users selects an account here
This also might be interesting for @desperateCoder @stefan-niedermann
- Dominant language
- Java
- Stars
- 74
- Forks
- 36
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 15
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from nextcloud/Android-SingleSignOn
-
0. Needs triage bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
nextcloud/Android-SingleSignOn#1081 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
nextcloud/Android-SingleSignOn#563 · 2 comments · 1 reaction ·
-
0. Needs triage bug
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
nextcloud/Android-SingleSignOn#894 · 1 reaction ·
-
0. Needs triage bug
Difficulty 4/5 3-5 days Newbie friendliness 35/100
nextcloud/Android-SingleSignOn#859 · 3 comments ·
-
0. Needs triage enhancement
Difficulty 4/5 3-5 days Newbie friendliness 35/100
All issues in nextcloud/Android-SingleSignOn
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
elastic/gradle-plugins#157 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
cryptomator/hub#497 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
johanhaleby/occurrent#1120 ·