Skip to content

refactor: Encapsulate aud config#4414

Merged
laurenceisla merged 2 commits into
PostgREST:mainfrom
mkleczek:jwt-aud-config-refactoring
Oct 22, 2025
Merged

refactor: Encapsulate aud config#4414
laurenceisla merged 2 commits into
PostgREST:mainfrom
mkleczek:jwt-aud-config-refactoring

Conversation

@mkleczek

@mkleczek mkleczek commented Oct 21, 2025

Copy link
Copy Markdown
Collaborator

This change is an initial step to change JWT aud configuration to regular expression (#4397). Exporting function
audMatchesCfg :: AppConfig -> Text -> Bool
from Config module allows changing the way how JWT aud is configured to be isolated and not affect code in Auth.JWT

This change is an initial step to change JWT aud configuration to regular expression.
Exporting function
audMatchesCfg :: AppConfig -> Text -> Bool
from Config module allows changing the way how JWT aud is configured to be isolated and not affect code in Auth.JWT
@mkleczek

Copy link
Copy Markdown
Collaborator Author

@laurenceisla I've seen you started reviewing #4397. I marked it as draft and would like to split it into smaller and easier to review PRs, that would also be less controversial.

@laurenceisla

Copy link
Copy Markdown
Member

I marked it as draft and would like to split it into smaller and easier to review PRs, that would also be less controversial.

Perfect, thanks for the heads-up.

@laurenceisla laurenceisla merged commit 6616110 into PostgREST:main Oct 22, 2025
32 of 33 checks passed
@mkleczek mkleczek deleted the jwt-aud-config-refactoring branch October 22, 2025 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants