Skip to content

Commit

Permalink
Bump FIRST_VERSION_REQUIRED_FOR_PROTOCOL_20
Browse files Browse the repository at this point in the history
  • Loading branch information
marta-lokhova committed Jan 29, 2024
1 parent 368063a commit 4d2b41f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/main/Config.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ Config::Config() : NODE_SEED(SecretKey::random())
LEDGER_PROTOCOL_MIN_VERSION_INTERNAL_ERROR_REPORT = 18;

OVERLAY_PROTOCOL_MIN_VERSION = 27;
OVERLAY_PROTOCOL_VERSION = 31;
OVERLAY_PROTOCOL_VERSION = 32;

VERSION_STR = STELLAR_CORE_VERSION;

Expand Down
2 changes: 1 addition & 1 deletion src/overlay/Peer.h
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ class Peer : public std::enable_shared_from_this<Peer>,
std::chrono::seconds(1);
static constexpr uint32_t FIRST_VERSION_SUPPORTING_FLOW_CONTROL_IN_BYTES =
28;
static constexpr uint32_t FIRST_VERSION_REQUIRED_FOR_PROTOCOL_20 = 31;
static constexpr uint32_t FIRST_VERSION_REQUIRED_FOR_PROTOCOL_20 = 32;

// The reporting will be based on the previous
// PEER_METRICS_WINDOW_SIZE-second time window.
Expand Down

5 comments on commit 4d2b41f

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging marta-lokhova/stellar-core/ban_ov_versions_before_32 = 4d2b41f into auto

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from MonsieurNicolas
at marta-lokhova@4d2b41f

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

marta-lokhova/stellar-core/ban_ov_versions_before_32 = 4d2b41f merged ok, testing candidate = f4b92b7

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = f4b92b7

Please sign in to comment.