Hi David, Thanks for your review. > On Dec 2, 2024, at 5:06 PM, David Mandelberg via Datatracker <noreply@xxxxxxxx> wrote: > > Reviewer: David Mandelberg > Review result: Ready > > Overall, looks good. I have one question which might be relevant to security, > but I'm not sure. > > Sections 5.2.1.1, 5.2.2.2, and 5.2.3.1 say "The BGP-LS-SPF Node NLRI Attribute > SPF Status TLV, Link NLRI Attribute SPF Status TLV, and Prefix NLRI Attribute > SPF Status TLV use the same TLV Type (1184). This implies that a BGP Update > will include a single NLRI." Is it possible to have multiple NLRI of different > types in the same update? Maybe it's worth adding a MUST NOT about that, and > adding something to section 7 about how to handle that case? We're not going to change this. Note that the Status TLVs are optional Thanks, Acee > > -- last-call mailing list -- last-call@xxxxxxxx To unsubscribe send an email to last-call-leave@xxxxxxxx
