Skip to content

[security]: add TruffleHog secret scanning#11

Open
pschork wants to merge 1 commit intomasterfrom
feature/sec-38-hardcoded-database-password-in-terraform-configuration
Open

[security]: add TruffleHog secret scanning#11
pschork wants to merge 1 commit intomasterfrom
feature/sec-38-hardcoded-database-password-in-terraform-configuration

Conversation

@pschork
Copy link
Copy Markdown

@pschork pschork commented May 7, 2026

Tracking issue

SEC-38

Why are the changes needed?

A historical Terraform file contained a hardcoded database password. That Terraform path has already been deleted from master, and the reported RDS cluster does not exist in us-west-2, but the repo did not have CI coverage to catch future committed secrets.

What changes were proposed in this pull request?

Adds a GitHub Actions workflow that runs TruffleHog on pull requests and master pushes. The scan uses --only-verified so CI fails on live, verified credentials while avoiding noise from historical/test strings.

How was this patch tested?

  • git diff --check origin/master...HEAD
  • Confirmed the reported file no longer exists on master
  • Confirmed flyteadmin-cluster does not exist in AWS RDS us-west-2
  • CI: TruffleHog Secret Scan passed

Labels

security

Setup process

N/A

Screenshots

N/A

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

N/A

Docs link

N/A

Link to Devin session: https://app.devin.ai/sessions/45c81ad103e3473498643cb74c3a8347
Requested by: @pschork

Adds a GitHub Actions workflow that scans PRs and master pushes for
hardcoded secrets using TruffleHog. Only verified (live) secrets trigger
failures to minimize noise from historical or test credentials.

Co-Authored-By: patrick <patrick@exa.ai>
@devin-ai-integration
Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@linear
Copy link
Copy Markdown

linear Bot commented May 7, 2026

SEC-38

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.

1 participant