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

Consider storing logged in status after page refresh #501

Open
IgorShadurin opened this issue Sep 13, 2023 · 2 comments
Open

Consider storing logged in status after page refresh #501

IgorShadurin opened this issue Sep 13, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@IgorShadurin
Copy link
Collaborator

Fairdrive's current behavior resets the logged in status whenever a user updates the page. It would be beneficial to reconsider this and possibly introduce a feature to retain the logged in status for a certain period. Here are a few things to determine:

  • Evaluating if storing the logged in status is secure for Fairdrive.
  • Ensuring the compatibility of all existing login methods such as username-password, Blossom, and Metamask.
  • Determining the ideal time period to store the login status.
  • Figuring out the best method to store this information (e.g., in localStorage/sessionStorage? Should it be encrypted or stored in raw format?)
  • Identifying any libraries that could assist in implementing this feature.

This improvement could significantly enhance the user experience while utilizing the app.

@tomicvladan @tfius

@IgorShadurin IgorShadurin added the enhancement New feature or request label Sep 13, 2023
@tomicvladan
Copy link
Collaborator

I don't think we can achieve this without sacrificing security. In order to keep user logged in after a page refresh/opening new page/ restarting browser etc., password must be stored somewhere locally. But it can't be encrypted, because there is no way to decrypt it. So if there were be any vulnerabilities in the app, then it would be possible to steal the password.

@tamas6
Copy link
Contributor

tamas6 commented Sep 20, 2023

related: #491

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

No branches or pull requests

3 participants