Skip to content

fix(cryptogen): improve error handling and add config parsing test#235

Open
sachin9058 wants to merge 1 commit into
hyperledger:mainfrom
sachin9058:fix/cryptogen-error-handling
Open

fix(cryptogen): improve error handling and add config parsing test#235
sachin9058 wants to merge 1 commit into
hyperledger:mainfrom
sachin9058:fix/cryptogen-error-handling

Conversation

@sachin9058

Copy link
Copy Markdown
Contributor

Type of change

  • Improvement
  • Test update

Description

This PR improves error handling in the cryptogen CLI and adds test coverage for configuration parsing failures.

Changes include:

  • removing ignored errors from fmt.Print/Println calls
  • improving error propagation with context in configuration parsing
  • standardizing error output to stderr with proper exit codes
  • adding a test to validate behavior on invalid configuration input

These changes improve reliability and make failures easier to diagnose without modifying core functionality.

Signed-off-by: Sachin Kumar <sachinkumar905846@gmail.com>
@sachin9058

Copy link
Copy Markdown
Contributor Author

Hi @mbrandenburger,

I’ve incorporated the suggested improvements and cleaned up the error handling and tests in this PR. Would appreciate your review when you get a chance.

Thanks!

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.

1 participant