-
Notifications
You must be signed in to change notification settings - Fork 436
feat(backend): Add Frontend API proxy helpers #7602
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
brkalow
wants to merge
23
commits into
main
Choose a base branch
from
brkalow/clerk-proxy-helper
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 all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
5ae1734
feat: Add Frontend API proxy helpers for @clerk/backend, @clerk/nextj…
brkalow b6e0da7
test: Add tests for Frontend API proxy helpers
brkalow 04ca626
test: Remove low-value Next.js proxy tests
brkalow dc1349e
feat: Add multi-domain support for Frontend API proxy
brkalow 93feba5
fix: Improve proxy header handling and redirect rewriting
brkalow 307789f
feat: Embed Frontend API proxy in clerkMiddleware for Express and Nex…
brkalow 77f9869
docs: Update Express frontendApiProxy examples to use enabled: true
brkalow 1c941f8
fix: Require explicit enabled: true for Express frontendApiProxy
brkalow 427dad7
fix: Use path boundary check for proxy path matching
brkalow fb6b536
fix: Normalize proxy path by removing trailing slashes
brkalow 7cf49a4
Merge branch 'main' into brkalow/clerk-proxy-helper
brkalow 48ab31e
fix: Handle x-forwarded-host and x-forwarded-proto as arrays
brkalow fe3eebc
refactor: Export proxy from @clerk/nextjs/server instead of /proxy su…
brkalow 82d436f
fix: Replace regex with loop for trailing slash removal
brkalow 7752813
refactor: Simplify stripTrailingSlashes using endsWith
brkalow 9c54ee1
refactor: Move Next.js proxy module into server directory
brkalow 7663c8c
fix: Fix CI failures for exports snapshot, test assertion, and missin…
brkalow b3eccb1
refactor: Remove DEFAULT_PROXY_PATH from @clerk/nextjs/server exports
brkalow 4dae2f9
fix: Fix import sorting in Express and clear mock between proxy tests
brkalow 05d1fd5
fix: Handle comma-separated x-forwarded-proto and x-forwarded-host he…
brkalow 78f01a3
fix: Strip query string from Express proxy path matching
brkalow eedc6d4
fix: Remove unused variable in proxy test
brkalow f6f4ab3
fix: Fix import sorting in Next.js proxy types and tests
brkalow 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| --- | ||
| '@clerk/backend': minor | ||
| '@clerk/nextjs': minor | ||
| '@clerk/express': minor | ||
| --- | ||
|
|
||
| Add Frontend API proxy support via `frontendApiProxy` option in `clerkMiddleware` | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| { | ||
| "main": "../dist/proxy.js", | ||
| "module": "../dist/proxy.mjs", | ||
| "types": "../dist/proxy.d.ts" | ||
| } |
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.
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.