advanced-security-compliance icon indicating copy to clipboard operation
advanced-security-compliance copied to clipboard

Code scanning fails even if there's a fix for an alert in the current branch

Open oliveiraleom opened this issue 4 years ago • 1 comments

Describe the bug Code scanning fails for new branch when there's an alert for master branch. The fix for the alert is in the new branch.

To Reproduce Steps to reproduce the behavior:

  1. Hard code a password in master and run the action against it. It will generate a code scanning alert
  2. Open a pull request with a fix for the hard coded password
  3. Action will fail in the new pull request even if there's a fix for the issue in there

Expected behavior Action should succeed in the new pull request as there's a fix for the code scanning alert

Screenshots Error in master: 001

Just master is an option here: 002

Action still fails in fix pull request : 003

Change that fixes the issue: 004

oliveiraleom avatar Aug 26 '21 11:08 oliveiraleom

@oliveiraleom Thanks for reporting this, I'll take a look at this shortly and get back to you with a patch.

GeekMasher avatar Aug 26 '21 14:08 GeekMasher

@oliveiraleom This issue is being closed and more to the new repo under another issue https://github.com/advanced-security/policy-as-code/issues/2

GeekMasher avatar Dec 14 '22 14:12 GeekMasher