Skip to content

chore: apply security best practices for CI #7066

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mohammed90
Copy link
Member

We need to pay close attention here to ensure nothing will break.

Supersedes / Closes #7065

@mohammed90 mohammed90 added the CI/CD 🔩 Automated tests, releases label Jun 13, 2025
Signed-off-by: Mohammed Al Sahaf <[email protected]>
Signed-off-by: Mohammed Al Sahaf <[email protected]>
@mohammed90
Copy link
Member Author

Good enough for me for now. It resolves many concerns. We can audit outbound calls here: https://app.stepsecurity.io/github/caddyserver/actions/runs. Resolving the scorecard generation for PRs can be tackled later.

@mohammed90 mohammed90 added the under review 🧐 Review is pending before merging label Jun 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD 🔩 Automated tests, releases under review 🧐 Review is pending before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant