Skip to content
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

Add 'IAP_VERIFY_AUDIENCE' option to en/disable audience check #450

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

jpassing
Copy link
Collaborator

@jpassing jpassing commented Jul 29, 2024

Add configuration option IAP_VERIFY_AUDIENCE:

  • The option defaults to true, meaning that the audience of IAP assertions is verified.
  • When set to false, the audience of IAP assertions is not verified and the option IAP_BACKEND_SERVICE_ID is ignored.

NB. The assertion's signature is always verified, regardless of IAP_VERIFY_AUDIENCE.

aebrahim and others added 2 commits July 24, 2024 23:15
Setting this for `IAP_BACKEND_SERVICE_ID` will skip verification of the
audience, allowing us to avoid the circular dependency from #324,
although it will still validate that the JWT audience contains the
correct project.

This option should be used by clients who have already verified that
the caller can only recieve traffic already validated by IAP and that
the project does not have any other backend services with less secure
IAP settings.
@jpassing jpassing merged commit 6c1e1ed into master Aug 1, 2024
6 checks passed
@jpassing jpassing deleted the pr/449 branch August 1, 2024 04:34
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.

2 participants