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

refactor: removing the arguments arg from the run python delete flow because it isn't used #29

Merged
merged 4 commits into from
Oct 2, 2024

Conversation

leoporoli
Copy link
Contributor

No description provided.

@leoporoli leoporoli force-pushed the lporoli/refactor-run-py-del branch from b06f7e6 to 16dcc16 Compare September 18, 2024 14:41
@leoporoli leoporoli requested a review from tedim52 September 18, 2024 15:13
Copy link

gitguardian bot commented Oct 2, 2024

⚠️ GitGuardian has uncovered 2 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
9423094 Triggered Generic Password cbc39b6 kontrol-service/docker-compose.yaml View secret
9423094 Triggered Generic Password cbc39b6 kontrol-service/dev-start-kk.sh View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@leoporoli leoporoli merged commit 0bd4f5c into main Oct 2, 2024
6 of 7 checks passed
@leoporoli leoporoli deleted the lporoli/refactor-run-py-del branch October 2, 2024 13:32
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