Skip to content

CI: Avoid persisting git credentials - #5073

Merged
Mynacol merged 1 commit into
RSS-Bridge:masterfrom
Mynacol:ci-persist-credentials
Aug 15, 2026
Merged

Mynacol merged 1 commit into
RSS-Bridge:masterfrom
Mynacol:ci-persist-credentials

Conversation

@Mynacol

@Mynacol Mynacol commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Following zizmor recommendations. The prhtmlupload workflow needs the credentials to push git commits at the end.

Following zizmor recommendations. The prhtmlupload workflow needs the
credentials to push git commits at the end.

Copilot AI 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.

Pull request overview

This PR hardens GitHub Actions workflows by preventing actions/checkout from persisting the repository token in the local git config, aligning with zizmor’s recommendation to reduce credential exposure risk during CI runs.

Changes:

  • Configure actions/checkout@v7 with persist-credentials: false in the test workflow.
  • Configure actions/checkout@v7 with persist-credentials: false in the PR HTML generation workflow (both PR and base checkouts).
  • Configure actions/checkout@v7 with persist-credentials: false across all jobs in the lint workflow.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/tests.yml Disables persisted git credentials for the test checkout step.
.github/workflows/prhtmlgenerator.yml Disables persisted git credentials for both PR and base repository checkouts.
.github/workflows/lint.yml Disables persisted git credentials for all lint job checkouts.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Mynacol
Mynacol merged commit cf86742 into RSS-Bridge:master Aug 15, 2026
13 checks passed
@Mynacol
Mynacol deleted the ci-persist-credentials branch August 15, 2026 22:35
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