Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix scorecards workflow runs #3303

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Conversation

dthaler
Copy link
Collaborator

@dthaler dthaler commented Feb 26, 2024

Description

https://github.com/microsoft/ebpf-for-windows/actions/workflows/scorecards-analysis.yml?query=branch%3Amain shows that the scorecards workflow is consistently failing in main.

As can be seen at
https://app.stepsecurity.io/github/microsoft/ebpf-for-windows/actions/runs/8052805389 we are hitting the errors because the score card action tries to access www.bestpractices.dev. This is the new name of
bestpractices.coreinfrastructure.org which was allowed (indeed if you try to go to that site in a browser, it now redirects to www.bestpractices.dev). This updates the ACL with the new domain name.

Testing

Covered by CI/CD.

Documentation

No impact.

Installation

No impact.

https://github.com/microsoft/ebpf-for-windows/actions/workflows/scorecards-analysis.yml?query=branch%3Amain
shows that the scorecards workflow is consistently failing in main.

As can be seen at
https://app.stepsecurity.io/github/microsoft/ebpf-for-windows/actions/runs/8052805389
we are hitting the errors because the score card action tries to access
www.bestpractices.dev.  This is the new name of
bestpractices.coreinfrastructure.org which was allowed (indeed if you
try to go to that site in a browser, it now redirects to
www.bestpractices.dev).  This updates the ACL with the new domain name.

Signed-off-by: Dave Thaler <[email protected]>
@saxena-anurag saxena-anurag added this pull request to the merge queue Feb 27, 2024
Merged via the queue into microsoft:main with commit cba8cc3 Feb 27, 2024
76 checks passed
@dthaler dthaler deleted the scorecard branch February 27, 2024 07:18
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.

3 participants