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

feat!: multiple services depending on same plugin implementation #61

Merged
merged 26 commits into from
Oct 10, 2024

Conversation

leoporoli
Copy link
Contributor

No description provided.

Copy link

gitguardian bot commented Oct 4, 2024

⚠️ GitGuardian has uncovered 4 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 59b20f4 kontrol-service/dev-start-kk.sh View secret
9423094 Triggered Generic Password 59b20f4 kontrol-service/docker-compose.yaml View secret
13106060 Triggered Generic High Entropy Secret 3da1351 ci/obd-demo.yaml View secret
13358409 Triggered Generic High Entropy Secret 3da1351 ci/obd-demo.yaml 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 force-pushed the lporoli/plugins-new-signature branch from 0696ef9 to 032b753 Compare October 4, 2024 20:08
@leoporoli leoporoli force-pushed the lporoli/plugins-new-signature branch from 67e29b8 to a99ecd6 Compare October 7, 2024 14:21
name: jsdelivr-api
annotations:
kardinal.dev.service/plugin-definition: |
- name: github.com/leoporoli/jsdelivr-api-plugin
Copy link
Contributor Author

Choose a reason for hiding this comment

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

will replace it in a next PR after merging plugins PRs

name: postgres-seed-plugin
annotations:
kardinal.dev.service/plugin-definition: |
- name: github.com/leoporoli/postgres-seed-plugin
Copy link
Contributor Author

Choose a reason for hiding this comment

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

will replace it in a next PR after merging plugins PRs

@leoporoli leoporoli requested a review from tedim52 October 7, 2024 19:54
Copy link
Contributor

@tedim52 tedim52 left a comment

Choose a reason for hiding this comment

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

LGTM! I think separating out the plugins into their own definition simplified the code a lot!

@leoporoli leoporoli merged commit 3fce223 into main Oct 10, 2024
6 of 7 checks passed
@leoporoli leoporoli deleted the lporoli/plugins-new-signature branch October 10, 2024 12:59
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