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 9:20 PM, Ben Campbell <[e-mail]> wrote:
> 
> 
> 
>> On Mar 19, 2019, at 1:43 PM, Adam Roach <[e-mail] <mailto:[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] <mailto:[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:

Or do we also need to remove the double-quotes?

> 
> 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