-
Notifications
You must be signed in to change notification settings - Fork 435
feat: Proxy handler support enabling My Account and My Org #2400
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
71 commits
Select commit
Hold shift + click to select a range
e9df6f0
feat: add my-account and my-org proxy
frederikprijck e8f0a10
add comment for search params
frederikprijck 0a4d50b
Use fetcher for proxy
frederikprijck 0a4aa37
Update proxy implementation
frederikprijck 0ed472c
Add test for POST
frederikprijck 748836e
Update comment
frederikprijck 4f179a3
chore: add local dev files to gitignore
tusharpandey13 a262c5b
Merge branch 'main' of https://github.com/auth0/nextjs-auth0 into fea…
tusharpandey13 c5d4680
chore: add local dev files to gitignore
tusharpandey13 b625525
Merge remote-tracking branch 'origin/chore/proxy' into feat/proxy-dpop
tusharpandey13 48ad253
debug: /me/ working, /org/ fails with dpop
tusharpandey13 1f65c29
feat: add my-account and my-org proxy
frederikprijck 758cd25
add comment for search params
frederikprijck 3655c4b
Use fetcher for proxy
frederikprijck c7b3db0
Update proxy implementation
frederikprijck d0c7414
Add test for POST
frederikprijck 1c1f970
Update comment
frederikprijck 3373af9
fix: use correct audience for my-account
frederikprijck 43c086c
chore: pass scope to handleProxy
frederikprijck 3b62aa8
chore: fix linter
frederikprijck 1a4b79c
chore: fix linter
frederikprijck 5e23701
fix: update cache in proxy handler
frederikprijck a970d6d
chore: move proxy tests to seperate file
frederikprijck b610c64
chore: revert changes
frederikprijck 2285a9f
chore: simplify default test data
frederikprijck 72a38c5
feat: add error handling and unit tests
frederikprijck a17bf17
fix: incorrect merge
frederikprijck 09459db
chore: fix linter
frederikprijck 13da0b2
chore: fix linter
frederikprijck b36eb86
chore: migrate tests to use msw
frederikprijck 4251af6
chore: add additional tests
frederikprijck 04113eb
chore: fix linter
frederikprijck af9c185
chore: add additional comment about removing authorization header
frederikprijck 707354c
chore: update test description
frederikprijck b27ba54
chore: update comment about how we handle session updates
frederikprijck de5f8a8
chore: fix incorrect merge
frederikprijck 1c53b69
chore: make handleProxy private
frederikprijck cd3a1c0
chore: use a single fetcher per audience
frederikprijck f9a2e3b
chore: explicitly define a list of headers to forward
frederikprijck c8e5ce4
chore: fix linter
frederikprijck e81c65c
Merge branch 'main' of https://github.com/auth0/nextjs-auth0 into fea…
tusharpandey13 7b8db87
Merge branch 'chore/proxy' of https://github.com/auth0/nextjs-auth0 i…
tusharpandey13 6af9e40
chore: update some docs
tusharpandey13 bd623e7
feat: fix cors, request body consumption and origin edge cases; make …
tusharpandey13 2f92754
chore: add exhaustive tests for handleProxy
tusharpandey13 7151a19
chore: reset examples/with-dpop to match main
tusharpandey13 f44dbb9
chore: remove files from examples/with-dpop that don't exist on main
tusharpandey13 42ce231
chore: update gitignore
tusharpandey13 302428d
fix: update path matching for proxy matcher
tusharpandey13 cdce2d8
chore: only clone req if dpop is on; fix redundant code
tusharpandey13 1e9698e
chore: lint
tusharpandey13 529ed74
chore: remove my-acc and my-org logic (will be added in a seperate PR)
tusharpandey13 3f416ef
feat: add back proxy test files
tusharpandey13 03e1bd6
feat: restore handleMyAccount and handleMyOrg methods and tests
tusharpandey13 10060ab
feat: add My Account and My Org proxy handlers with bug fixes
tusharpandey13 a1c483e
feat: restore comprehensive proxy handler tests using /me endpoint
tusharpandey13 304a536
fix: resolve proxy handler test failures for /me endpoint
tusharpandey13 ba940f1
chore: add docs; remove redundant header from allowlist; update tests
tusharpandey13 f77293d
chore: make handlePreflight fwd response status from upstream; use sc…
tusharpandey13 c62b8bd
fix: ensure fetcher's getAccessToken can access the tokenSetSideEffec…
frederikprijck d6e5719
chore: no need to bind the getAccessToken handler
frederikprijck 047a270
chore: revert unnecessary changes to simplify diff
frederikprijck 4a77cd4
chore: update tests
tusharpandey13 c4bda8c
chore: update tests; fix buggy targeturl logic
tusharpandey13 25b1669
chore: avoid including test files in dist directory (#2396)
frederikprijck bd93260
Merge branch 'main' into feat/my-account-my-org-clean
tusharpandey13 048550a
remove unnecessary duplex options
frederikprijck 6d34679
Replace auth0-scope header with standard scope header
tusharpandey13 53871e1
Add tests for basePath handling in auth handler
tusharpandey13 d3ca61a
fix: remove exact equality checks for matcher for /me and /my-org
tusharpandey13 6f8d7be
Merge branch 'main' into feat/my-account-my-org-clean
tusharpandey13 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
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.
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.
If the page makes a
My Accountrequest (updateProfilein this case) during page render, and that page is linked using the Next.js<Link>component, it could trigger an actual API request during prefetch in production.I think we should either fix this or document this.