Skip to content

Ajout de tfsec et checkov dans la CI#100

Open
Scttpr wants to merge 4 commits intomainfrom
ci/sast-tools
Open

Ajout de tfsec et checkov dans la CI#100
Scttpr wants to merge 4 commits intomainfrom
ci/sast-tools

Conversation

@Scttpr
Copy link
Copy Markdown
Member

@Scttpr Scttpr commented Dec 15, 2025

🤔 Pourquoi ?

Dans la perspective de l'ouverture du programme et du renforcement de nos CI, j'ajoute une couche de SAST specifique a Terraform.

🍰 Comment ?

Un workflow Github qui run tfsec et checkov.

🚨 À vérifier

  • Le commit message est rédigé en anglais
  • Le titre de la PR et sa présente description sont en français

Comment thread .github/workflows/_terraform-module.yml Fixed
Comment thread .github/workflows/_terraform-module.yml Fixed
@Scttpr Scttpr requested a review from leo-naeka December 15, 2025 15:29
@leo-naeka
Copy link
Copy Markdown
Collaborator

Je ne suis pas certain qu'ajouter tfsec soit la bonne option long terme : le projet est délaissé au profit de trivy et n'est quasi plus maintenu.

Je serais d'avis d'ajouter l'action trivy plutôt. Qu'en penses-tu ?
Je regarde checkov, a priori RAS.

Par contre à quoi sert le script ? Est-ce qu'on n'en ferait pas une action pre-commit plutôt ?

@Scttpr
Copy link
Copy Markdown
Member Author

Scttpr commented Dec 19, 2025

Je ne suis pas certain qu'ajouter tfsec soit la bonne option long terme : le projet est délaissé au profit de trivy et n'est quasi plus maintenu.

Je serais d'avis d'ajouter l'action trivy plutôt. Qu'en penses-tu ? Je regarde checkov, a priori RAS.

Par contre à quoi sert le script ? Est-ce qu'on n'en ferait pas une action pre-commit plutôt ?

Yes je regarde pour migrer vers Trivy, le script oui pour le precommit, je modifie aussi

@Scttpr
Copy link
Copy Markdown
Member Author

Scttpr commented Jan 12, 2026

@leo-naeka j'ai change pour Trivy et j'ai supprime le script inutile. Je ne suis pas certain que le precommit soit necessaire sur du SAST, c'est plutot a la CI d'avoir ce role ? Je l'integre si tu n'es pas d'accord evidemment.

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.

3 participants