Skip to content

Only grant IAP access to the console for new users#3167

Merged
gbrodman merged 1 commit into
google:masterfrom
gbrodman:onlyConsoleBackendIap
Jul 22, 2026
Merged

Only grant IAP access to the console for new users#3167
gbrodman merged 1 commit into
google:masterfrom
gbrodman:onlyConsoleBackendIap

Conversation

@gbrodman

@gbrodman gbrodman commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Currently we grant users IAP_SECURED_WEB_APP_USER_ROLE access to the
entire project upon creation if there is no preexisting group set up for
them to use. These users only need to be able to access the console, so
we should restrict it to the console.

If there is a group (which we do have in our prod + sandbox
configuration) then we just add them to that group instead and that
group controls all the privileges. So this is a no-op for our setup.

We prioritize the group -- if both the group and the IAP service ID are
specified, we'll still just add them to the group

b/534931170


This change is Reviewable

Currently we grant users IAP_SECURED_WEB_APP_USER_ROLE access to the
entire project upon creation if there is no preexisting group set up for
them to use. These users only need to be able to access the console, so
we should restrict it to the console.

If there is a group (which we do have in our prod + sandbox
configuration) then we just add them to that group instead and that
group controls all the privileges. So this is a no-op for our setup.

We prioritize the group -- if both the group and the IAP service ID are
specified, we'll still just add them to the group

b/534931170
@gbrodman
gbrodman force-pushed the onlyConsoleBackendIap branch from 24c7b19 to 33ad3e1 Compare July 21, 2026 17:22
@gbrodman
gbrodman requested a review from weiminyu July 22, 2026 15:57

@weiminyu weiminyu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@weiminyu reviewed 17 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on gbrodman).

@gbrodman
gbrodman added this pull request to the merge queue Jul 22, 2026
Merged via the queue into google:master with commit d3a1d07 Jul 22, 2026
16 checks passed
@gbrodman
gbrodman deleted the onlyConsoleBackendIap branch July 22, 2026 19:26
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