Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

ENG-171 Run GitGuardian in CI as part of Unified CI in Integrations #25

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

sethidden
Copy link
Contributor

If you add the GitGuardian app globally to the org, it only scans code when a PR is opened. This doesn't protect us against the case where someone just creates a branch with plaintext credentials, but doesn't create a PR.

Adding GitGuardian scans as a GitHub action with a "on push" trigger will make GitGuardian scan all code that appears in the repo, rather than just code which is part of a PR.

To make this work, you will need to add GITGUARDIAN_API_KEY GitHub secret in each repository. Organization-wide GitHub secrets apply only on private repositories, so to cover all repos reliably you'll need to create a regular per-repo secret, not an org-wide one.

@sethidden sethidden requested a review from a team as a code owner November 15, 2023 14:03
If you add the GitGuardian app globally to the org, it only scans code
when a PR is opened. This doesn't protect us against the case where
someone just creates a branch with plaintext credentials, *but doesn't
create a PR*.

Adding GitGuardian scans as a GitHub action with a "on push" trigger
will make GitGuardian scan all code that appears in the repo, rather
than just code which is part of a PR.

To make this work, you will need to add GITGUARDIAN_API_KEY GitHub
secret in each repository. Organization-wide GitHub secrets apply only on private
repositories, so to cover all repos reliably you'll need to create a regular
per-repo secret, not an org-wide one.
@sethidden sethidden merged commit 1f4f69f into main Nov 15, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants