Skip to content

fix(general): exclude ecr-public:GetAuthorizationToken from credentials exposure - #7654

Open
ROunofF wants to merge 1 commit into
bridgecrewio:mainfrom
ROunofF:fix-ckv-aws-107-287-ecr-public-auth-token
Open

fix(general): exclude ecr-public:GetAuthorizationToken from credentials exposure#7654
ROunofF wants to merge 1 commit into
bridgecrewio:mainfrom
ROunofF:fix-ckv-aws-107-287-ecr-public-auth-token

Conversation

@ROunofF

@ROunofF ROunofF commented Aug 21, 2026

Copy link
Copy Markdown

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Description

CKV_AWS_107 and CKV_AWS_287 flag ecr-public:GetAuthorizationToken on "Resource": "*" as credentials exposure, but the action does not support resource-level permissions, so there is no way to remediate the finding. This mirrors the existing exclusion for ecr:GetAuthorizationToken added in #1819 .

Fixes # (issue)

New/Edited policies (Delete if not relevant)

Description

Include a description of what makes it a violation and any relevant external links.

Fix

How does someone fix the issue in code and/or in runtime?

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my feature, policy, or fix is effective and works
  • New and existing tests pass locally with my changes

…ls exposure

CKV_AWS_107 and CKV_AWS_287 flag ecr-public:GetAuthorizationToken on
"Resource": "*" as credentials exposure, but the action does not support
resource-level permissions, so there is no way to remediate the finding.
This mirrors the existing exclusion for ecr:GetAuthorizationToken added
in bridgecrewio#1819.
@ROunofF
ROunofF force-pushed the fix-ckv-aws-107-287-ecr-public-auth-token branch from 1ef5e95 to 427ffa9 Compare August 21, 2026 00:44
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.

1 participant