Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

Just to make sure I looked at all binary.LittleEndian calls in Dela: can you confirm that the following line is NOT a problem, as it only uses the data, but not the key?

https://github.com/dedis/dela/blob/master/core/validation/simple/simple.go#L65

Just to make sure I looked at all binary.LittleEndian calls in Dela: can you confirm that the following line is NOT a problem, as it only uses the data, but not the key?

https://github.com/dedis/dela/blob/master/core/validation/simple/simple.go#L65

I'm not sure if that line is related to the key-value store in question, isn't it related to https://github.com/dedis/dela/blob/master/core/store/store.go?

the only places I found that handle the key of the Bolt store were in the same file as the change of this commit, and the key was only created, and used to retrieve data but not cast back to an uint64

moved the PR to draft, as I'm still testing though

tests look good so far (> 300 votes), but I had to test on https://github.com/dedis/dela/tree/fix-bbolt for d-voting, since d-voting is not working with the current head

Closing this for the new PRs