Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Fdawgs
Copy link
Member

@Fdawgs Fdawgs commented Sep 23, 2025

Added an extra conditional to the automerge job to check Dependabot PRs are coming from the same repo and not a fork.
Never seen this happen but Dependabot/GitHub developers aren't infallible and bugs can occur, so is an extra security layer.

Checklist

@github-actions github-actions bot added the github actions Github actions related label Sep 23, 2025
@Uzlopak
Copy link
Contributor

Uzlopak commented Sep 23, 2025

I think this is already covered in the code of the action. I think i had also claimed a cve for this.

@Fdawgs
Copy link
Member Author

Fdawgs commented Sep 23, 2025

I think this is already covered in the code of the action. I think i had also claimed a cve for this.

Not that I can see? Your CVE was around imitating Dependabot by changing the associated username and email for a malicious commit.

Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

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

LGTM

@Uzlopak
Copy link
Contributor

Uzlopak commented Sep 23, 2025

Allrighty ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github actions Github actions related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants