Election-Tech-Initiative / Election-Tech-Initiative/electionguard

🐞 Specification 1.1 for extended crypto_hash does not match implementation.

Open
#290 0 comments 0 reactions 0 assignees View on GitHub
bug triage
Dominant language
Python
Stars
872
Forks
131
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

Apparently we discussed this in https://github.com/microsoft/electionguard-python/issues/279

and decided to use

Q̅ = H(Q, commitment_hash)

but the 1.1 spec still has (top of page 15):

(3.B) Q = H(Q, K).

### Expected Behavior

Spec should be changed to match the implementation.

### Steps To Reproduce

_No response_

### Environment

```markdown
- OS:
```

### Anything else?

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.