On Fri, Aug 09, 2024 at 10:28:20PM +0200, Claudio Jeker wrote:
> This diff introduces a 'min-version' config option for rtr sessions.
> The session can only be established if the negotiated version with the
> cache is larger or equal to min-version. The default is 0 which is the old
> RFC6810 spec.
> 
> Now on top of this introduce RTR_DEFAULT_VERSION which is not set to 1 for
> RFC8210. If people want to try ASPA and the draft-8210bis code you need to
> set min-version 2. Then the code will negotiate and require version 2
> which includes the ASPA PDU.
> 
> I lightly tested this and it seems to work fine. There is currently no
> support to clear a RTR session to pick up the new config. I will add
> something for that later.
> 
> IMO defaulting to version 1 is currently the best option until the dust of
> RTRv2 settled.

That all makes sense to me. Diff reads fine.

ok tb