fix(message-hash): account for `timestamp` and align with specs
danisharora099 opened this issue · comments
This is a bug report
Problem
Based on the specs: https://github.com/vacp2p/rfc-index/blob/main/waku/standards/core/14/message.md#deterministic-message-hashing, the current message-hash algorithm did not take into account the timestamp.
The test vectors mentioned in the spec also varied from the one in the implementation.
Proposed Solutions
- Update algorithm to account for
timestamp - Update test vectors
Notes
Original message: https://discord.com/channels/1110799176264056863/1111541029309534249/1233338075191709768