Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

INT-7619: fix duplicated Audit Config IAM Policy relationship key #579

Merged
merged 2 commits into from
Apr 6, 2023

Conversation

gastonyelmini
Copy link
Contributor

Fix duplicated Audit Config IAM Policy relationship key

Google's auditLogConfig.exemptedMembers response is the following:

auditLogConfig [
  {
    logType: 'ADMIN_READ',
    exemptedMembers: [ 'user:[email protected]' ]
  },
  {
    logType: 'DATA_READ',
    exemptedMembers: [ 'user:[email protected]' ]
  }
]

This is generating a duplicated key error when a member is included in one or more actions.

Proposed solution

Merge relationships additionalProperties logType and avoid adding it twice to the jobState.

@gastonyelmini gastonyelmini requested a review from a team as a code owner March 30, 2023 19:19
@ndowmon ndowmon added patch Increment the patch version when merged release Create a release when this pr is merged labels Apr 6, 2023
@ndowmon ndowmon merged commit 88c805c into main Apr 6, 2023
@ndowmon ndowmon deleted the INT-7619-fix-duplicated-key branch April 6, 2023 16:04
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v2.21.1 🚀

@j1-internal-automation j1-internal-automation added the released This issue/pull request has been released. label Apr 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
patch Increment the patch version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants