Skip to content

Disable persisted checkout credentials in e2e workflow - #1115

Merged
brunoborges merged 4 commits into
mainfrom
copilot/fix-code-scanning-alert-124
Jul 14, 2026
Merged

Disable persisted checkout credentials in e2e workflow#1115
brunoborges merged 4 commits into
mainfrom
copilot/fix-code-scanning-alert-124

Conversation

CopilotAI commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Description:
Addresses the code scanning finding on the e2e workflow by removing credential persistence from the flagged checkout step.

  • Workflow hardening
    • Adds persist-credentials: false to the actions/checkout@v7 step in setup-java-set-default within .github/workflows/e2e-versions.yml.
    • Aligns this job with the existing secure checkout pattern already used elsewhere in the workflow.
  • Scope
    • Limits the change to the single step identified by the alert.
    • No production code or test logic changes.
- name: Checkoutuses: actions/checkout@v7with:
persist-credentials: false

Related issue:
Code scanning alert #124

Check list:

  • Ran npm run check locally (format, lint, build, test) and all checks pass.
  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

CopilotAI changed the title [WIP] Fix code scanning alert #124Disable persisted checkout credentials in e2e workflowJul 14, 2026
CopilotAI requested a review from brunoborgesJuly 14, 2026 19:20
@brunoborges
brunoborges marked this pull request as ready for review July 14, 2026 20:04
@brunoborges
brunoborges requested a review from a team as a code ownerJuly 14, 2026 20:04
CopilotAI review requested due to automatic review settings July 14, 2026 20:04

CopilotAI 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 the .github/workflows/e2e-versions.yml GitHub Actions workflow by disabling persisted checkout credentials for the setup-java-set-default job’s actions/checkout@v7 step, aligning it with the existing secure checkout pattern already used elsewhere in the workflow.

Changes:

  • Add with: persist-credentials: false to the Checkout step in the setup-java-set-default job.
Show a summary per file
FileDescription
.github/workflows/e2e-versions.ymlDisables persisted checkout credentials in the setup-java-set-default job to address the flagged workflow security finding.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Low

@brunoborges
brunoborges merged commit 80c368b into mainJul 14, 2026
423 checks passed
@brunoborges
brunoborges deleted the copilot/fix-code-scanning-alert-124 branch July 14, 2026 20:11
Sign up for freeto 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

@brunoborges