Skip to content

Fix CodeQL security alerts - #46

Merged
AevumDecessus merged 1 commit into
fragforce:devfrom
AevumDecessus:fix-codeql-alerts
Jul 1, 2026
Merged

Fix CodeQL security alerts#46
AevumDecessus merged 1 commit into
fragforce:devfrom
AevumDecessus:fix-codeql-alerts

Conversation

@AevumDecessus

Copy link
Copy Markdown
Contributor

Fix 3 CodeQL security findings from PR #45.

Changes

Medium Severity

  • Add missing permissions: contents: read to validate-release workflow
  • Remove exception detail from healthz endpoint to prevent information disclosure

Proactive Security

  • Add CodeQL workflow to scan PRs and pushes to dev/main
  • Scan Python, JavaScript, and GitHub Actions workflows
  • Catches security issues before they reach main

Related

  • Fixes CodeQL alerts blocking PR Release 1.0.0 #45
  • Does not address the high severity sonar.yaml alert (already following documented safe pattern)

Add missing permissions block to validate-release workflow (medium severity).

Remove exception detail from healthz endpoint to prevent information disclosure (medium severity).

Add CodeQL workflow to scan PRs to dev and main for Python, JavaScript, and GitHub Actions.
@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2026

Copy link
Copy Markdown

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@AevumDecessus
AevumDecessus merged commit b79e7ab into fragforce:dev Jul 1, 2026
7 of 10 checks passed
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