-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add Option to Register if Registration fails in first attempt #143
Comments
@guy34code can i work on this issue? |
You're welcome to work on this issue. Registering creates a public/private key pair in the NDN keychain for that user's username/identity. This needs to be one of the things that is checked. In case there's some other problem, but the keys are created successfully, they should be deleted in case the user decides to enter a different username or namespace. |
@jrandallclark now i m working on this issue. |
@jrandallclark there is no option for signup? and replaced register with login. what is database to signup and signin? |
SQLite? |
Sqlite is used by NDN Keychain (via jNDN) to store the keys. But you should not have to interact with Sqlite directly iirc. When the app is started for the first time (i.e. no data from previous run), the page it presents is the registration page. There is no sign out once you register (since there is no remote server, everything is in this app) so sign-in does not make sense. |
sir @agawande what are the essential keys to register into app? |
This is an NDN app. So the domain is a routeable NDN namespace. Currently, only local communication where both devices are connected to the same router is supported, so anything that doesn't include the string "npChat" will work. As your account is tied directly to your phone at the moment, the password is not currently used for anything. The fix for this issue would be to check that the user's identity and public/private key pair are successfully created in the NDN keychain and the username is stored in shared preferences. The login boolean value is also in shared preferences. If any part of registering fails, the user should be automatically sent back to the registration/login screen. Additionally, any partially finished work needs to be deleted in the event of a failure. You can see where the NDN keychain is being accessed in the BackgroundJobs class. |
Summary:
When you fail to register for first time for any reasons, there is not an option to register again, you just have to clear data or reinstall apk to Register again.
Steps to reproduce:
If your registration fails and home screen appears, then there is no option to for registration again.
Device and Android version:
Model: Xiaomi MiA1
Android: 9.0(Pie)
ROM: Stock ROM
Screen-shots:
Would you like to work on the issue?
Yes, I would like to work on this issue.
The text was updated successfully, but these errors were encountered: