Skip to content

Commit

Permalink
RTR server does not support v2 just yet. (#281)
Browse files Browse the repository at this point in the history
This PR limits the protocol version supported by the RTR server to 1 or less.

While we technically support version 2, the ASPA PDU format has not yet been
agreed upon and may still change. So rather than have people deploy
potentially broken servers, we disable the version.
  • Loading branch information
partim authored Jan 17, 2024
1 parent 2a95512 commit 3814337
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/rtr/server.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,11 @@ use super::state::State;
/// The maximum protocol version we support.
///
/// We support all protocol versions from 0 up to and including this value.
const MAX_VERSION: u8 = 2;
///
/// While the server technically supports version 2 as well, the format of the
/// ASPA PDU has not yet been agreed upon. Rather than possibly deploying
/// broken servers, we only announce support for version 0 or 1 for now.
const MAX_VERSION: u8 = 1;

//============ Traits ========================================================

Expand Down

0 comments on commit 3814337

Please sign in to comment.