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

feat(iam): conditions MTA-5686 #4532

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat(iam): conditions MTA-5686 #4532

wants to merge 6 commits into from

Conversation

ldecarvalho-doc
Copy link
Contributor

No description provided.

@ldecarvalho-doc ldecarvalho-doc added type: new content New pages or categories status: tech review Waiting for technical review priority: high PRs that include documentation on breaking changes to products and features status: doc review Waiting for doc team review labels Mar 3, 2025
@ldecarvalho-doc ldecarvalho-doc self-assigned this Mar 3, 2025
ldecarvalho-doc and others added 2 commits March 4, 2025 11:08
Co-authored-by: Nathanael Demacon <[email protected]>
Co-authored-by: Rowena Jones <[email protected]>
Co-authored-by: Néda <[email protected]>
Copy link
Contributor

@crlptl crlptl left a comment

Choose a reason for hiding this comment

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

Tremendous work!

@ldecarvalho-doc ldecarvalho-doc added the do not merge PR that shouldn't be merged before a specific date (eg release) label Mar 5, 2025
@Oursin
Copy link
Contributor

Oursin commented Mar 5, 2025

It could be useful to have a link to the String functions as we already have one to the Date/Time functions since we use some in the examples: https://github.com/google/cel-spec/blob/master/doc/langdef.md#string-functions

### Multiple policies
If multiple policies with different conditions apply to the same principal, the presence of a single policy with met conditions (or no conditions) will override any denying rules from other policies, allowing the action to be taken.

For example, if you set up a policy that grants access to a resource only on Monday while another policy grants access only on Tuesday, the action will still be permitted on Monday.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd clarify this by saying that the action can be performed on both Monday and Tuesday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge PR that shouldn't be merged before a specific date (eg release) priority: high PRs that include documentation on breaking changes to products and features status: doc review Waiting for doc team review status: tech review Waiting for technical review type: new content New pages or categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants