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

Apply changes from https://github.com/casper-network/casper-node/pull/5089 #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gRoussac
Copy link
Contributor

@gRoussac gRoussac commented Feb 5, 2025

Apply changes from casper-network/casper-node#5089

  • Renamed BinaryRequest to Command
  • changed BinaryResponseAndRequest so it holds responses requests raw bytes instead of derived structure

Missing : Update of example files resources/examples/*.bin

Getting request_id from BinaryResponseAndRequest request is a bit strange, I would have expected not to have to extract the request id from the request but have it in BinaryResponseHeader

@gRoussac gRoussac marked this pull request as ready for review February 6, 2025 15:39
@gRoussac
Copy link
Contributor Author

gRoussac commented Feb 6, 2025

@igor-casper @zajko Is there a process to update files in resources/examples/ ?

thanks

@gRoussac gRoussac force-pushed the binary_response_changes branch from bf2f46a to 3fa4b84 Compare February 6, 2025 15:59
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