Skip to main content

Description Option for the Port Control Protocol (PCP)
RFC 7220

Yes

(Ted Lemon)

No Objection

(Adrian Farrel)
(Gonzalo Camarillo)
(Jari Arkko)
(Joel Jaeggli)
(Martin Stiemerling)
(Richard Barnes)
(Spencer Dawkins)
(Stewart Bryant)

Note: This ballot was opened for revision 04 and is now closed.

(Ted Lemon; former steering group member) Yes

Yes (for -04)

                            

(Adrian Farrel; former steering group member) No Objection

No Objection (for -04)

                            

(Barry Leiba; former steering group member) No Objection

No Objection (2014-02-17 for -04)
It looks like Pete's DISCUSS point about the NUL termination is resolved; that works for me.

(Benoît Claise; former steering group member) No Objection

No Objection (2014-02-17 for -04)
Thanks for addressing the OPS-DIR feedback.

(Brian Haberman; former steering group member) No Objection

No Objection (2014-02-19 for -04)
I have no objection to the publication of this document, but I do support:

1. Pete's (and Stephen's) points on the null character

2. Pete's Comments on implementation details in Section 3

3. Stephen's Comment on needing better/more privacy discussion related to the DESCRIPTION field

(Gonzalo Camarillo; former steering group member) No Objection

No Objection (for -04)

                            

(Jari Arkko; former steering group member) No Objection

No Objection (for -04)

                            

(Joel Jaeggli; former steering group member) No Objection

No Objection (for -04)

                            

(Martin Stiemerling; former steering group member) No Objection

No Objection (for -04)

                            

(Pete Resnick; former steering group member) (was Discuss) No Objection

No Objection (2014-02-21)
Thanks for addressing my issues. Here's what's left:

Section 2:

- I still believe that "The description text MUST NOT be null terminated" is probably wrong, but at least unnecessary. The document now says, "the description text is not null terminated", and puts the requirement on the receiver to do the right thing. This sounds like you're still requiring the sender to scan for the terminating null. But I don't think that anything bad will happen because of this.

Section 3, Paragraphs 4 & 5 try to put limits on the internal buffer sizes of the server, which is not appropriate. Please replace *both* paragraphs 4 & 5 with the following:

   Because of the UDP payload limit of 1100 octets, the Length of the
   Description MUST NOT exceed 1016 octets.  The suggested maximum
   length is 128 octets.  If a PCP client includes a DESCRIPTION option
   with a length exceeding the maximum length supported by the PCP
   server, only the portion of the Description field fitting that
   maximum length is stored by the PCP server and returned to the PCP
   client in the response.

(Richard Barnes; former steering group member) No Objection

No Objection (for -04)

                            

(Sean Turner; former steering group member) No Objection

No Objection (2014-02-20 for -04)
Agree with Stephen - should not leak privacy data is pretty lame

(Spencer Dawkins; former steering group member) No Objection

No Objection (for -04)

                            

(Stephen Farrell; former steering group member) No Objection

No Objection (2014-02-17 for -04)

- "o Etc." is not a good bullet in a PS

- "SHOULD NOT be used to leak privacy-related
information" is pretty lame really isn't it? Who do
you know that would set out to "leak privacy related
information"? Perhaps giving some better guidance,
e.g. say to not use customer IDs or names or PII or
addresses or locations, would be easy and worthwhile? 

- On Pete's null terminated issue and the resulting
mail threads. I assume that the authors know that
mid-string null's have been used as part of deliberate
attacks against PKI? So there are security reasons as
well as interop reasons for not wanting nulls anywhere
within strings. Not worth a discuss, but maybe worth
a note.

(Stewart Bryant; former steering group member) No Objection

No Objection (for -04)