Mercurial > hg > ietf
diff draft-ietf-sidrops-avoid...txt @ 18:d8dde47d0a7f
as submitted?
| author | Henry Thompson <ht@markup.co.uk> |
|---|---|
| date | Mon, 19 Oct 2026 13:42:27 +0100 |
| parents | 4ca654e435db |
| children |
line wrap: on
line diff
--- a/draft-ietf-sidrops-avoid...txt Tue Jun 16 15:08:37 2026 +0100 +++ b/draft-ietf-sidrops-avoid...txt Mon Oct 19 13:42:27 2026 +0100 @@ -1,1 +1,54 @@ https://datatracker.ietf.org/doc/draft-ietf-sidrops-avoid-rpki-state-in-bgp/reviewrequest/24139/login/ +Document: draft-ietf-sidrops-avoid-rpki-state-in-bgp-11 +Title: Guidance to Avoid Carrying RPKI Validation States in BGP Path Attributes +Reviewer: Henry S. Thompson +Review Date: 2025-06-17 + +*Summary* + +As far as I can tell as a non-specialist, the presentation of this +proposed BCP is in good shape. There is one minor suggestion and some +editorial nits below. + +*Minor point* + +1. + + ... "states leak through to EGBP speakers." Would it be possible to + _briefly_ mention here at least one known way in which such leakage + can occur in a conformant router? That way operators of 'good' + routers may realise this document is relevant to them. + +*Nits* + +1. + + In this phrase + + "Annotating ... Attribute carried across EBGP session (based on validation + states) means that BGP UPDATE messages have to be sent to every + neighbor when the aforementioned validation states" + + there's a minor grammatical problem. I would suggest the following + change + + "... Attribute carried across an EBGP session (based on validation + states) means that BGP UPDATE messages have to be sent to every + neighbor when the aforementioned validation states" + +4. + + "BGP communities" -> "BGP Communities" [here and in 4 other places below] + + "hold similar" -> "hold in a similar way" + +4.1 + + "a large amounts" -> "large amounts" + +4.1.2 + + "caused by operators to signal validation states" -> + "caused by operators signalling validation states" + +
