-
Notifications
You must be signed in to change notification settings - Fork 15.8k
New lesson: Session Management in Practice #29814
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
Open
mao-sz
wants to merge
26
commits into
TheOdinProject:main
Choose a base branch
from
mao-sz:node-revamp-auth-compare
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
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
a0fcca9
Write intro
mao-sz e007ec4
Write intro to comparison
mao-sz 78653cc
Write content on stateless caveats
mao-sz 9fe0b67
Recommend authentication approach for remaining TOP projects
mao-sz 44af6d0
Add assignment and knowledge check contents
mao-sz 737f383
Expand on invalidation<->stale data
mao-sz 9c59f9a
Fix typo
mao-sz ed494e4
Remove ineffective codespell:ignore comment
mao-sz 1bcc6b8
Emphasise context of lesson's comparison
mao-sz 6b6998f
Rephrase invalidating Odin token example
mao-sz 8d529ec
Fix subsection heading levels
mao-sz e7cf5f2
Rephrase initial Odin demotion example
mao-sz 5c6ddc8
Streamline wording for access/refresh tokens
mao-sz 62708ab
Fix mythological inaccuracy
mao-sz 6f0fa79
Fix grammar
mao-sz 804c5b5
Merge branch 'main' into node-revamp-auth-compare
mao-sz ebdc6ce
Merge branch 'main' into node-revamp-auth-compare
mao-sz 6a1aea3
Rename lesson
mao-sz 2c1198c
Update intro/lesson overview with updated scope
mao-sz 3e0b841
Streamline verbiage
mao-sz ac33332
Add section on third-party cookies
mao-sz bf646cc
Write section on reverse proxies
mao-sz ff8f2d8
Add assignment/KC questions about reverse proxy with Netlify/Vercel
mao-sz 37d7190
Streamline verbiage
mao-sz 547494a
Remove cryto assignment
mao-sz b61d2b6
Rephrase LO items to be topic overviews than learning objectives
mao-sz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very minor grammar check. But where you said "(though this is of course sometimes an desirable security feature, like with many banking websites)." should say "sometimes a desirable security feature"
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch! Though you're looking at an outdated commit - the latest version has that fixed (fixed in 3e0b841).
There will have been quite a few changes since the commit you had looked at.