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

fix: handle invalid secret formats in parse_secret #326

Merged
merged 6 commits into from
May 3, 2024

Conversation

J0
Copy link
Contributor

@J0 J0 commented Mar 20, 2024

What kind of change does this PR introduce?

Fix #324 handles cases where an empty string or an invalid string format is passed in.

  • Write test

@J0 J0 changed the title fix: add catch all clause fix: handle invalid secret formats in parse_secret Mar 20, 2024
@J0 J0 force-pushed the j0/handle_non_sha256_and_md5_parse_secret branch from 05373ce to 50f9206 Compare May 2, 2024 13:05
@J0 J0 marked this pull request as ready for review May 2, 2024 14:26
@J0 J0 requested review from abc3 and chasers May 2, 2024 14:26
@abc3 abc3 merged commit b960662 into main May 3, 2024
2 checks passed
@abc3 abc3 deleted the j0/handle_non_sha256_and_md5_parse_secret branch May 3, 2024 19:27
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.

Extend Helpers.parse_secret/2 to handle nil or empty secrets
2 participants