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.

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

Fix the order of data passed to the host. #320

Merged
merged 1 commit into from
Jul 19, 2023
Merged

Fix the order of data passed to the host. #320

merged 1 commit into from
Jul 19, 2023

Conversation

abizjak
Copy link
Contributor

@abizjak abizjak commented Jul 19, 2023

The node expects data before the signature.

https://github.com/Concordium/concordium-node/blob/main/concordium-consensus/src/Concordium/Scheduler.hs#L1454

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

The node expects data before the signature.
@abizjak abizjak requested a review from DOBEN July 19, 2023 13:24
@abizjak abizjak merged commit 61dfab4 into main Jul 19, 2023
126 checks passed
@abizjak abizjak deleted the fix-order branch July 19, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants