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

Hash-onion config should be validated #5813

Closed
shuse2 opened this issue Sep 15, 2020 · 0 comments · Fixed by #5818
Closed

Hash-onion config should be validated #5813

shuse2 opened this issue Sep 15, 2020 · 0 comments · Fixed by #5818
Assignees
@shuse2
Labels
framework type: bug
Milestone
Sprint 28

Comments

@shuse2
Copy link
Collaborator

shuse2 commented Sep 15, 2020

Expected behavior

When foring.delegates of the config exist, hashOnion and the content of hashOnion should be required, and hashes should be at least length of 1.

Actual behavior

Currently there is no validation of hashOnion

Steps to reproduce

Add empty hashes to the config and start the application

Which version(s) does this affect? (Environment, OS, etc...)

5.0.0-alpha.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@shuse2 shuse2

Labels
framework type: bug
Projects
None yet
Milestone
Sprint 28
Development

Successfully merging a pull request may close this issue.

Add hash onion config validation - Closes #5813 LiskArchive/lisk-sdk
1 participant
@shuse2