Skip to content

fix(CICD): Disable Gitleaks usage in CICD#160

Merged
lukeroantreeONS merged 1 commit intomainfrom
disable_docker_gitleaks
Mar 27, 2026
Merged

fix(CICD): Disable Gitleaks usage in CICD#160
lukeroantreeONS merged 1 commit intomainfrom
disable_docker_gitleaks

Conversation

@lukeroantreeONS
Copy link
Copy Markdown
Contributor

📌 Newest gitleaks release requires username/password for docker image; disable in pipeline.

✨ Summary

Newest gitleaks release requires username/password for docker image; disable in pipeline as GitGuardian offers same protection.
Note: there should be a follow-up PR adding in the official Gitleaks Action, but out of scope at this moment due to need for v1.0.0 release, which this issue is blocking.

📜 Changes Introduced

  • fix - remove gitleaks from CICD

✅ Checklist

Please confirm you've completed these checks before requesting a review.

  • Code passes linting with Ruff
  • Security checks pass using Bandit
  • API and Unit tests are written and pass using pytest
  • Terraform files (if applicable) follow best practices and have been validated (terraform fmt & terraform validate)
  • DocStrings follow Google-style and are added as per Pylint recommendations
  • Documentation has been updated if needed

🔍 How to Test

Check the CI/Lint Action runs successfully in this PR

…for setup, no longer works in CICD, so disabling as GitGuardian offers same protection
@lukeroantreeONS lukeroantreeONS requested a review from a team as a code owner March 27, 2026 16:01
@github-actions github-actions Bot added the bug Something isn't working label Mar 27, 2026
Copy link
Copy Markdown
Contributor

@frayle-ons frayle-ons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lukeroantreeONS lukeroantreeONS merged commit 6089271 into main Mar 27, 2026
4 checks passed
@lukeroantreeONS lukeroantreeONS deleted the disable_docker_gitleaks branch March 27, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants