You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Commit e020ce4 by @aashaikh added asn as the value used in conjunction with repeat-n to prepend the AS path with, but did not specify behaviour if asn is present but repeat-n is not.
Sane options are:
Refuse to accept asn without repeat-n
Prepend asn once
Do nothing
Option 1 could be implemented using when statement, but now it's probably too late for it may break existing configurations.
Option 2 essentially gives repeat-n two default values: 0 when asn is empty, and 1 otherwise: not great theoretically, though understandable practically. Whether this is a problem is subjective.
Option 3 seems safe unless 2 is already popular in the wild, which is hard for me to tell.
In any case, something needs to be done to prevent implementations divergence.
The text was updated successfully, but these errors were encountered:
Commit e020ce4 by @aashaikh added
asn
as the value used in conjunction withrepeat-n
to prepend the AS path with, but did not specify behaviour ifasn
is present butrepeat-n
is not.Sane options are:
asn
withoutrepeat-n
asn
onceOption 1 could be implemented using
when
statement, but now it's probably too late for it may break existing configurations.Option 2 essentially gives
repeat-n
two default values:0
whenasn
is empty, and1
otherwise: not great theoretically, though understandable practically. Whether this is a problem is subjective.Option 3 seems safe unless 2 is already popular in the wild, which is hard for me to tell.
In any case, something needs to be done to prevent implementations divergence.
The text was updated successfully, but these errors were encountered: