Skip to content

feat(env): add support for JWKS discovery endpoints#53

Merged
awaseem merged 6 commits into
mainfrom
aliwaseem/func-601-support-loading-jwks-from-url-in-supabaseserver
May 8, 2026
Merged

feat(env): add support for JWKS discovery endpoints#53
awaseem merged 6 commits into
mainfrom
aliwaseem/func-601-support-loading-jwks-from-url-in-supabaseserver

Conversation

@awaseem
Copy link
Copy Markdown
Contributor

@awaseem awaseem commented May 7, 2026

What kind of change does this PR introduce?

Feature: Support for JWKS urls

  • I added a inline variable that stores the remote cache from Jose when called once, Thinking here was that a subsequent serverless runtimes make requests in the warm pool we don't need to call it over and over again to reduce latency and rely on the underlying lib for caching
  • Added some tests with the help of claude

@awaseem awaseem requested review from kallebysantos and tomaspozo and removed request for kallebysantos May 7, 2026 20:39
Comment thread docs/environment-variables.md Outdated
@awaseem awaseem marked this pull request as draft May 7, 2026 20:56
@awaseem
Copy link
Copy Markdown
Contributor Author

awaseem commented May 7, 2026

Moving to draft because change in direction

@awaseem awaseem marked this pull request as ready for review May 8, 2026 13:43
Comment thread docs/environment-variables.md
Comment thread README.md Outdated
Comment thread docs/environment-variables.md Outdated
Copy link
Copy Markdown
Member

@kallebysantos kallebysantos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me,
We just need to make sure to this env is not referenced as default exposed.

@awaseem awaseem requested review from kallebysantos and tomaspozo May 8, 2026 15:10
Copy link
Copy Markdown
Member

@tomaspozo tomaspozo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A small one proposing to support local dev urls

Comment thread src/core/resolve-env.ts Outdated
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 8, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@supabase/server@53

commit: 5487d92

@awaseem awaseem merged commit 45d677a into main May 8, 2026
7 checks passed
@awaseem awaseem deleted the aliwaseem/func-601-support-loading-jwks-from-url-in-supabaseserver branch May 8, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants