-
Notifications
You must be signed in to change notification settings - Fork 10
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
P2P: Add additional info to handshake message #977
Comments
@heifner Can you consider how we would want to do this if breaking changes are on the table anyway as a v2.0.0 item or as a network protocol change in v1.1.0? |
Current
struct handshake_message_v2 : handshake_message {
block_num_type head_num = 0;
block_id_type head_id;
block_timestamp_type head_block_time;
block_timestamp_type fork_head_block_time;
fc::time_point last_vote_received;
fc::time_point last_trx_received;
fc::time_point last_blk_received;
size_t trx_in_progress_size_bytes = 0;
bool block_log = true;
block_num_type start_block = 0;
block_num_type end_block = 0;
};
|
@heifner can you add information on how each of these fields would be useful? |
Replaced by #1083 |
The
os
field of handshake currently contains one ofosx
,linux
,win32
,other
.Repurpose this field to include additional information about the peer.
|vt={time_point in seconds}
- last vote received time|ts={trx_in_progress_size kilo-bytes}
- can be zero if no speculative trx currently queued for execution|tt={time_point in seconds}
- last trx received timeThe text was updated successfully, but these errors were encountered: