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 Nonce Default in --interactive Mode #24

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

pcaversaccio
Copy link
Owner

@pcaversaccio pcaversaccio commented Mar 13, 2025

🕓 Changelog

Fixes #22. This PR removes the unnecessary nonce override since it is already provided via the CLI. Additionally, fallback values are added for the version and response parameters in case curl fails silently. These changes ensure the script runs smoothly under all conditions.

Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio self-assigned this Mar 13, 2025
@pcaversaccio pcaversaccio added the bug 🐛 Something isn't working label Mar 13, 2025
@pcaversaccio pcaversaccio added the refactor/cleanup ♻️ Code refactorings and cleanups label Mar 13, 2025
@pcaversaccio pcaversaccio merged commit 66643a9 into main Mar 13, 2025
5 of 6 checks passed
@pcaversaccio pcaversaccio deleted the fix/nonce-overrides branch March 13, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working refactor/cleanup ♻️ Code refactorings and cleanups
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Nonce Defaulting to 0 in --interactive Mode
1 participant