Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normalize provided private key string to the correct PEM format #22

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

ykitamura-mdsol
Copy link
Contributor

Storing a private key string in an environment variable could be challenging with the correct PEM format (having newline characters).

With this PR, the provided private key string will be normalized to the correct PEM format.

Copy link

gitguardian bot commented Dec 23, 2024

⚠️ GitGuardian has uncovered 1 secret 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 secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
14950534 Triggered RSA Private Key 2506152 src/pem_format.rs View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Revoke and rotate the secret.

🦉 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.

@ykitamura-mdsol ykitamura-mdsol force-pushed the feature/normalize_pem_key_format branch from 7fb9b3b to c9c465c Compare December 23, 2024 03:00
@ykitamura-mdsol ykitamura-mdsol force-pushed the feature/normalize_pem_key_format branch from c9c465c to 2506152 Compare December 23, 2024 03:02
@ykitamura-mdsol ykitamura-mdsol merged commit 54d05f3 into main Dec 24, 2024
4 checks passed
@ykitamura-mdsol ykitamura-mdsol deleted the feature/normalize_pem_key_format branch December 24, 2024 07:43
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.

2 participants