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

Build real authentication #353

Open
nikolagsiderov opened this issue Apr 7, 2021 · 0 comments
Open

Build real authentication #353

nikolagsiderov opened this issue Apr 7, 2021 · 0 comments

Comments

@nikolagsiderov
Copy link
Owner

nikolagsiderov commented Apr 7, 2021

Currently, the authentication is purely basic hardcoded sessions and cookies.
Research and check out how it is done now.
We create sessions with headers such as: CurrentUserID, CurrentHomeID, etc.
If these are present, then we assume user is 'authenticated'.

As you finish your research on the current code base, you will agree this needs to be dropped.

OK, build a real authentication flow.

Current structure of the platform:
Web application -> API layer -> Infrastructure & services -> ORM

Do we need authentication both on web and api layer?
Let's say I try signing in:
After I enter my credentials, will they encoded/encrypted somehow, while being on the server side?
Will the web layer send the encrypted credentials to api layer to actually authenticate?
Should the api layer have a certificate? Or is this for another story?

We want to drop the current 'authentication', but preserve the ability to have global variables, such as CurrentUserID and CurrentHomeID. Check out BaseController, where they are being initialized, in the web layer.

Propose your solution here, after doing a proper research, for me to check out. We will do a call on this discussion here before we proceed doing the changes.

@nikolagsiderov nikolagsiderov changed the title Write down **real** authentication Build a **real** authentication Apr 7, 2021
@nikolagsiderov nikolagsiderov changed the title Build a **real** authentication Build real authentication Apr 7, 2021
@nikolagsiderov nikolagsiderov self-assigned this Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

2 participants