Re: [sipcore] Adam Roach's Discuss on draft-ietf-sipcore-reason-q850-loc-06: (with DISCUSS and COMMENT)

[Ben Campbell <[e-mail]>]

> On Mar 19, 2019, at 1:43 PM, Adam Roach <[e-mail]> wrote:
> 
> On 3/15/19 1:28 PM, Adam Roach wrote:
>> On 3/15/19 1:23 PM, Adam Roach wrote:
>>> On 3/11/19 3:23 AM, [e-mail] wrote:
>>>> Hi,
>>>> Thank you for your comments. I went through the comments and here are my answers and proposals to the comments made.
>>>> Sorry If some of you get the mail twice, since I answered in another way already.
>>>> 
>>>> 1.  Discuss:
>>>> §4 ABNF
>>>> Have incorporated the proposal.
>>> 
>>> 
>>> I'm going to clear my discuss
>>> 
>> 
>> Based on Paul Kyzivat's comment on the SIPCORE mailing list, I'm going to hold off and talk to Ben before I clear. Sorry for the confusion.
>> 
> 
> Based on Paul's comment and on Ben's reminder that SIP fields are generally case-insensitive (both on the SIPCORE mailing list), I think the solution here is to simply remove all of the instances of "%s" in the BNF I sent. Sorry for the extra round-trip here.
> 

Like the following? If so, I can drop this into an RFC Editor note:

OLD:
reason-extension =/ isup-cause-location
isup-cause-location =  "location" EQUAL isup-location-value

   isup-location-value =
      %s"U" /      ; for 0 0 0 0 user
      %s"LPN" /    ; for 0 0 0 1 private network serving the local user
      %s"LN" /     ; for 0 0 1 0 public network serving the local user
      %s"TN" /     ; for 0 0 1 1 transit network
      %s"RLN" /    ; for 0 1 0 0 public network serving the remote user
      %s"RPN" /    ; for 0 1 0 1 private network serving the remote user
      %s"LOC-6" /  ; for 0 1 1 0 spare
      %s"INTL" /   ; for 0 1 1 1 international network
      %s"LOC-8" /  ; for 1 0 0 0 spare
      %s"LOC-9" /  ; for 1 0 0 1 spare
      %s"BI" /     ; for 1 0 1 0 network beyond interworking point
      %s"LOC-11" / ; for 1 0 1 1 spare
      %s"LOC-12" / ; for 1 1 0 0 reserved for national use
      %s"LOC-13" / ; for 1 1 0 1 reserved for national use
      %s"LOC-14" / ; for 1 1 1 0 reserved for national use
      %s"LOC-15"   ; for 1 1 1 1 reserved for national use

                       Figure 1: isup-cause-location
NEW:
reason-extension =/ isup-cause-location
isup-cause-location =  "location" EQUAL isup-location-value

   isup-location-value =
      "U" /      ; for 0 0 0 0 user
      "LPN" /    ; for 0 0 0 1 private network serving the local user
      "LN" /     ; for 0 0 1 0 public network serving the local user
      "TN" /     ; for 0 0 1 1 transit network
      "RLN" /    ; for 0 1 0 0 public network serving the remote user
      "RPN" /    ; for 0 1 0 1 private network serving the remote user
      "LOC-6" /  ; for 0 1 1 0 spare
      "INTL" /   ; for 0 1 1 1 international network
      "LOC-8" /  ; for 1 0 0 0 spare
      "LOC-9" /  ; for 1 0 0 1 spare
      "BI" /     ; for 1 0 1 0 network beyond interworking point
      "LOC-11" / ; for 1 0 1 1 spare
      "LOC-12" / ; for 1 1 0 0 reserved for national use
      "LOC-13" / ; for 1 1 0 1 reserved for national use
      "LOC-14" / ; for 1 1 1 0 reserved for national use
      "LOC-15"   ; for 1 1 1 1 reserved for national use

                       Figure 1: isup-cause-location