Skip to content

SONARJAVA-7186 Set contents rights to 'write' in the unified dogfooding workflow - #6346

Merged
aurelien-coet-sonarsource merged 2 commits into
masterfrom
ac/fix-unified-dogfooding
Oct 14, 2026
Merged

aurelien-coet-sonarsource merged 2 commits into
masterfrom
ac/fix-unified-dogfooding

Conversation

@aurelien-coet-sonarsource

@aurelien-coet-sonarsource aurelien-coet-sonarsource commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary by Gitar

  • Permissions:
    • Changed contents permission from read to write for the Build for Shadow Scan and Run IRIS jobs in unified-dogfooding.yml

This will update automatically on new commits.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Set contents rights to 'write' in the unified dogfooding workflow SONARJAVA-7186 Set contents rights to 'write' in the unified dogfooding workflow Oct 14, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

SONARJAVA-7186

@gitar-bot

gitar-bot Bot commented Oct 14, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🔴 High risk · Granting the Build for Shadow Scan workflow contents: write could let a compromised or misused job modify repository contents.

Grants contents: write permission to the Build for Shadow Scan and Run IRIS jobs in unified-dogfooding.yml, changing it from read. No issues found.

Verification

Functional validation No results

Rules No rules evaluated

Auto-approval Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@BartLucien BartLucien left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@sonarqube-next

Copy link
Copy Markdown
Contributor

@aurelien-coet-sonarsource
aurelien-coet-sonarsource merged commit 00aac78 into master Oct 14, 2026
19 checks passed
@aurelien-coet-sonarsource
aurelien-coet-sonarsource deleted the ac/fix-unified-dogfooding branch October 14, 2026 09:10
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