Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Messages] Preserve original message time #254

Closed
wants to merge 1 commit into from
Closed

[Messages] Preserve original message time #254

wants to merge 1 commit into from
+57 −2

Commits on May 10, 2022

  1. [Messages] Preserve original message time 

    Fixed an issue where the CCN would overwrite the original "time"
field of the message by the time at which the message is confirmed
on-chain. This issue could result in inversions of order between
messages if two consecutive messages arrive in the opposite order
on the node in charge of confirming messages.

Added a migration script to reset the message time field using
the content.time field. The migration script does not update
forgotten messages as their content is null. The extra effort
to update these messages is not worth it given that it requires
to parse the whole chain data history for a few tens of messages.
    @odesenfans
    odesenfans committed May 10, 2022
    Configuration menu
    Copy the full SHA
    a450a14 View commit details
    Browse the repository at this point in the history