-
Notifications
You must be signed in to change notification settings - Fork 89
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 authorization #182
Comments
Hey there Arfey, I will look into this over the weekend. |
good luck 👍 if u'll have any questions feel free to ask. Also, i recommend u to do this task step by step and push small changes to review |
Hey there Arfey, had to be off for this for a good while. And starting to get back on it. Is the login page suppose to be the first page seen when running the app? |
Hi. In my opinion, login page should not be as start page, because u can get index page without authz. Also when u run create-aio-app and after that make run u open login page but u don't have a user for login. |
True, I got the login is page ready. Just the page right now. No auth or postgres related data. Will be working on the auth side thru the week after work. |
@sanyen are you still working on this? |
Yes, on hold for personal reasons. Will be resuming this weekend. |
@amartya-dev feel free to take this over. Think I've bitten off a bit more than I can handle right now. |
Ok, I think my approach would be this:
Also, I would request the maintainers to include this in hacktoberfest |
https://aiohttp-security.readthedocs.io/en/latest/
user info - postgres
session data - redis
The text was updated successfully, but these errors were encountered: