Skip to content
This repository has been archived by the owner on Apr 9, 2022. It is now read-only.

Migrate Authentication to Cookies #155

Open
tko22 opened this issue Sep 4, 2018 · 1 comment
Open

Migrate Authentication to Cookies #155

tko22 opened this issue Sep 4, 2018 · 1 comment
Assignees
@n3a9 @caiyingshen
Labels
good first issue Good for newcomers Priority: Low we can live without this

Comments

@tko22
Copy link
Member

tko22 commented Sep 4, 2018 •
edited
Loading

It's currently using sessionStorage, but you can't access it in server rendering, which limits the power of Next.js. Migrating to storing keys in cookies would allow the server to view authentication keys there. Heck, maybe we can change up auth as a whole and make it more secure. that's super low priority though.

Example: vercel/next.js#5821
@tko22 tko22 added good first issue Good for newcomers Priority: Low we can live without this labels Sep 4, 2018
@tko22
Copy link
Member Author

tko22 commented Dec 27, 2018

This would allow for server side fetching in resources, allowing for lazy loading in next.js

@tko22 tko22 mentioned this issue Dec 27, 2018
Open
@tko22 tko22 assigned caiyingshen, jackieo5023 and n3a9 and unassigned jackieo5023 Jul 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@n3a9 n3a9

@caiyingshen caiyingshen

Labels
good first issue Good for newcomers Priority: Low we can live without this
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@n3a9 @caiyingshen @tko22 @jackieo5023