-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
New lesson: Session-based authentication #29082
Open
MaoShizhong
wants to merge
22
commits into
TheOdinProject:main
Choose a base branch
from
MaoShizhong:node-revamp-sessions
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
New lesson: Session-based authentication #29082
MaoShizhong
wants to merge
22
commits into
TheOdinProject:main
from
MaoShizhong:node-revamp-sessions
Commits on Nov 6, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 3c3a0fd - Browse repository at this point
Copy the full SHA 3c3a0fdView commit details
Commits on Nov 9, 2024
-
Configuration menu - View commit details
-
Copy full SHA for e212eb6 - Browse repository at this point
Copy the full SHA e212eb6View commit details -
'Persisting logins' was a glorified intro anyway
Configuration menu - View commit details
-
Copy full SHA for 4d23f47 - Browse repository at this point
Copy the full SHA 4d23f47View commit details -
Configuration menu - View commit details
-
Copy full SHA for a2d7e44 - Browse repository at this point
Copy the full SHA a2d7e44View commit details -
Configuration menu - View commit details
-
Copy full SHA for 657de49 - Browse repository at this point
Copy the full SHA 657de49View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0027857 - Browse repository at this point
Copy the full SHA 0027857View commit details
Commits on Nov 10, 2024
-
Configuration menu - View commit details
-
Copy full SHA for ca2d4f9 - Browse repository at this point
Copy the full SHA ca2d4f9View commit details -
Explain session options in app.js
Reordered setup code blocks for easier content flow. Decided to introduce the session store and secret explanations here instead of their own sections later on (not really needed there).
Configuration menu - View commit details
-
Copy full SHA for 0cbc997 - Browse repository at this point
Copy the full SHA 0cbc997View commit details
Commits on Nov 13, 2024
-
Configuration menu - View commit details
-
Copy full SHA for d767546 - Browse repository at this point
Copy the full SHA d767546View commit details -
Rearrange session options description
New order allows for a more natural way of explaining how express-session populates req.session. That can be explained at the start and options explained to reflect that, instead of explaining options in a bit of a black hole then provide the context later in the lesson.
Configuration menu - View commit details
-
Copy full SHA for 79ef33e - Browse repository at this point
Copy the full SHA 79ef33eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2c0da9b - Browse repository at this point
Copy the full SHA 2c0da9bView commit details -
Configuration menu - View commit details
-
Copy full SHA for e5bfc45 - Browse repository at this point
Copy the full SHA e5bfc45View commit details -
Made more sense to talk about logging out after logging in, then talk about password hashing afterwords.
Configuration menu - View commit details
-
Copy full SHA for 584b2a4 - Browse repository at this point
Copy the full SHA 584b2a4View commit details -
Use subsections for better content organisation and linking
Configuration menu - View commit details
-
Copy full SHA for 1869145 - Browse repository at this point
Copy the full SHA 1869145View commit details -
Configuration menu - View commit details
-
Copy full SHA for 45bf391 - Browse repository at this point
Copy the full SHA 45bf391View commit details -
Configuration menu - View commit details
-
Copy full SHA for 03da52f - Browse repository at this point
Copy the full SHA 03da52fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 77d56e1 - Browse repository at this point
Copy the full SHA 77d56e1View commit details -
"Session-based authentication" is a more appropriate title for the lesson than just "Sessions", given that sessions are not exclusively used for auth purposes.
Configuration menu - View commit details
-
Copy full SHA for 973251c - Browse repository at this point
Copy the full SHA 973251cView commit details -
Demonstrate NODE_ENV conditional cookie options
Makes sense to showcase them directly rather than just via text example after the fact
Configuration menu - View commit details
-
Copy full SHA for b9b0b1e - Browse repository at this point
Copy the full SHA b9b0b1eView commit details
Commits on Nov 17, 2024
-
Force hash verification to run every POST /login
Prevent timing attack
Configuration menu - View commit details
-
Copy full SHA for 9c5f8a2 - Browse repository at this point
Copy the full SHA 9c5f8a2View commit details
Commits on Nov 19, 2024
-
Configuration menu - View commit details
-
Copy full SHA for db5787a - Browse repository at this point
Copy the full SHA db5787aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9a8148c - Browse repository at this point
Copy the full SHA 9a8148cView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.