Re: [babel] Document Shepherd review of draft-ietf-babel-source-specific-04

[Juliusz Chroboczek <[e-mail]>]

Thanks a lot for the review, Donald.

We're slightly incoherent here:

  - Section 7 says that multiple sub-TLVs in a single Update MUST NOT be
    sent and the whole TLV SHOULD be ignored;
  - Section 7 also says that a sub-TLV in a retraction MUST NOT be sent,
    while Section 5 says that such a sub-TLV MUST be ignored on reception.

(You appear to have missed the bit in Section 5, which I cannot blame you
for -- it needs repeating in Section 7).

The reason for the inconsistence is that an implementation must check for
the presence of the sub-TLV in any case, so the requirement to discard
retractions with the sub-TLV puts no extra onus on implementations.  On
the other hand, checking for the case of multiple occurrences of the
sub-TLV puts a small cost on implementations, but one that's probably
negligible.

I'm definitely going to update Section 7 to repeat the requirement in
Section 5.

I think you're right, that the SHOULD about multiple sub-TLVs should be
a MUST, but I want to sleep over it and check the implementation.  Toke,
do you have an opinion?

-- Juliusz