-
Notifications
You must be signed in to change notification settings - Fork 467
fix: txn cookie accumulation #2748
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
Merged
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
16ae32f
fix: prevent __txn_* cookie accumulation via value-prefix encoding an…
Piyush-85 e29c7e0
docs: prevent __txn_* cookie accumulation via value-prefix encoding a…
Piyush-85 b6a3296
fix: remove prefetch flag, simplify txn cookie eviction to single-pha…
Piyush-85 541a86b
fix: lint fix
Piyush-85 b30ec24
fix: fix txn cookie eviction limit and add session size warning
Piyush-85 4dd65a7
fix: consolidate 431 docs and inline transaction cookie cleanup
Piyush-85 537de26
fix: extract eviction logic into evictOldestTransactionCookies privat…
Piyush-85 27119c4
fix: addressing coderabbit review comments
Piyush-85 d1d12f0
fix: address code review findings on txn cookie accumulation PR
Piyush-85 e7a44b3
fix: failing test with passwordless nonce
Piyush-85 a6af9f6
Merge branch 'main' into fix/txn-accumulation
Piyush-85 7867c52
fix: detect Sec-Purpose prefetch;prerender in isNonNavigationalReques…
Piyush-85 4bc926b
fix: delete stale __session__N chunks deterministically to prevent re…
Piyush-85 e6193cf
fix: dedup MFA step-up access tokens to prevent session-cookie growth…
Piyush-85 28a8c23
key MFA step-up token dedup on audience & scope to preserve different…
Piyush-85 27d2f7f
fix: key MFA token dedup on requested scope and tighten session cooki…
Piyush-85 f97c52a
fix: addressing review comments
Piyush-85 9564d85
fix: prevent __txn_* cookie accumulation via value-prefix encoding an…
Piyush-85 5037282
docs: prevent __txn_* cookie accumulation via value-prefix encoding a…
Piyush-85 5e68900
fix: remove prefetch flag, simplify txn cookie eviction to single-pha…
Piyush-85 d2a75cf
fix: lint fix
Piyush-85 2bcffd2
fix: fix txn cookie eviction limit and add session size warning
Piyush-85 591e8c2
fix: consolidate 431 docs and inline transaction cookie cleanup
Piyush-85 f964ab4
fix: extract eviction logic into evictOldestTransactionCookies privat…
Piyush-85 ef6b6af
fix: addressing coderabbit review comments
Piyush-85 5985220
fix: address code review findings on txn cookie accumulation PR
Piyush-85 95baeee
fix: failing test with passwordless nonce
Piyush-85 69a4ad6
fix: detect Sec-Purpose prefetch;prerender in isNonNavigationalReques…
Piyush-85 1a5dfa7
fix: address txn-accumulation review findings
Piyush-85 7ef34be
docs: update 401 to 204 on prefetch restriction
Piyush-85 33aae38
fix: addressing review comments
Piyush-85 169026d
fix: improving test coverage on stateless session
Piyush-85 a8484c4
fix: addressing review comments
Piyush-85 d3dffb8
fix: merge fix/txn-accumulation into fix/session-cookie-accumulation
Piyush-85 dda2ea6
fix: address session-cookie accumulation review findings
Piyush-85 47f5732
fix: addressing few nit comments
Piyush-85 f34b491
fix: prevent session-cookie accumulation (stale __session__N chunks, …
Piyush-85 6b38a2e
Merge remote-tracking branch 'origin/main' into fix/txn-accumulation
Piyush-85 c5c51f7
fix: addressing review comments
Piyush-85 5c2bdc0
fix: clamp scope/audience, split eviction warn, document dedup diverg…
Piyush-85 3bcf846
Merge branch 'main' into fix/txn-accumulation
Piyush-85 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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.