Election-Tech-Initiative / Election-Tech-Initiative/electionguard-cpp
🐞 Throw an error if manifest contains sequence_order of zero
- Dominant language
- C++
- Stars
- 26
- Forks
- 25
- PR merge metrics
- No merged PRs in 30d
Description
### Is there an existing issue for this?
- [X] I have searched the existing issues
### Current Behavior
If the manifest contains any zero sequence_orders then it allows them through, but that fails later in the process.
### Expected Behavior
_No response_
### Steps To Reproduce
_No response_
### Environment
```markdown
- OS:
```
### Anything else?
_No response_
Contributor guide
Research direction
No file or test is named. Start by locating the manifest parsing and validation entry point, then trace how sequence_order values are handled. Add coverage for a zero value and verify that manifests containing one are rejected before later processing.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- cryptography
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100