Mapping and Interworking of Diversion Information between Diversion and History-Info Headers in the Session Initiation Protocol (SIP)
draft-mohali-diversion-history-info-07

[Ballot comment]
The authors have included text in the Introduction that is very satisfactory and I have cleared my Discuss.

I should have liked to also see some note in the Abstract, but this is a non-blocking Comment that I leave to the authors to consider. Suitable text might be...

  This document does not recommend further implementation or
  deployment of the Diversion header.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

5) The period at the end of the line is invalid ABNF according to the BAP tool:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

6) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ... index=1.1.1,
                  ^ remove  this

7) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

8) Section 7.1: remove last "," ?

  History-Info:
  ... ;index=1.1.1.1,
                    ^ remove this

9) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

5) The period at the end of the line is invalid ABNF according to the BAP tool:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

6) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ... index=1.1.1,
                  ^ remove  this

7) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

8) Section 7.1: remove last "," ?

  History-Info:
  ... ;index=1.1.1.1,
                    ^ remove this

9) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period at the end of the line is invalid ABNF according to the BAP tool:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ... index=1.1.1,
                  ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ... ;index=1.1.1.1,
                    ^ remove this

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period is invalid ABNF:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ... index=1.1.1,
                  ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ... ;index=1.1.1.1,
                    ^ remove this

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period is invalid ABNF:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ... index=1.1.1,
                  ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ... ;index=1.1.1.1,
                    ^ remove

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period is invalid ABNF:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ;index=1,
  ;index=1.1,
  ; index=1.1.1,
                                                    ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ; index=1,
  ;index=1.1,
  ;index=1.1.1,
  ;index=1.1.1.1,
                                                        ^ remove

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period is invalid ABNF:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ;index=1,
  ;index=1.1,
  ; index=1.1.1,
                                                    ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ; index=1,
  ;index=1.1,
  ;index=1.1.1,
  ;index=1.1.1.1,
                                                        ^ remove

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period is invalid ABNF:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ;index=1,
  ;index=1.1,
  ; index=1.1.1,
                                                    ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ; index=1,
  ;index=1.1,
  ;index=1.1.1,
  ;index=1.1.1.1,
                                                        ^ remove

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

3) Section 3.1: r/header:/header. ?

4) Section 7.4: There are some line breaks missing between paragraphs.

[Ballot discuss]
1) The period is invalid ABNF:

  cause-param = "cause" EQUAL Status-Code.
                                          ^ remove

2) Section 3.1: I don't think the final "," is supposed to be there:

  History-Info:
  ;index=1,
  ;index=1.1,
  ; index=1.1.1,
                                                    ^ remove  this

2) General: Shouldn't 2119 language be used in this document (and need a normative reference to 2119 and the boilerplate)?  Especially in Section 5, 6, and 7.4?

3) Section 7.1: remove last "," ?

  History-Info:
  ; index=1,
  ;index=1.1,
  ;index=1.1.1,
  ;index=1.1.1.1,
                                                        ^ remove

4) Please add pointers to the security considerations in [RFC4244] and [RFC5806].  Something like: The security considerations in [RFC4244] and [RFC50806] apply.

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

[Ballot discuss]
1) Section 3.1: I don't think the final "," is supposed to be there:

History-Info:
  ;index=1,
  ;index=1.1,
  ; index=1.1.1,
                                                    ^ remove  this

[Ballot comment]
1) Section 1.2: r/deploye/deploy

2) Section 3: Remove line:

OLD:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,

  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off


NEW:

  diverting_user2_addr; reason="user-busy"; counter=1; privacy=full,
  diverting_user1_addr; reason="unconditional"; counter=1;
  privacy=off

[Ballot discuss]
Discuss for consideration by the IESG.

What is our position on an RFC that encourages the use of an alternative to an IETF specification? While I understand the benefit of interworking being specificed, wouldn't it be better to have

- a clear statement that the new implementation and deployment of
  the Diversion header is strongly discouraged

- a stronger note that this work is intended to enable the migration
  from non-standard implementations and deployment toward IETF
  specification-based implementations and deployment.