Re: [stir] [Technical Errata Reported] RFC8224 (6519)

[Christer Holmberg <[e-mail]>]

Ok, thanks for the memory refresh.

Regards,

Christer

-----Original Message-----
From: Marc Petit-Huguenin <[e-mail]> 
Sent: keskiviikko 7. huhtikuuta 2021 22.25
To: Roman Shpount <[e-mail]>; Christer Holmberg <[e-mail]>
Cc: [e-mail]; [e-mail]; [e-mail]; [e-mail]; [e-mail]; [e-mail]; [e-mail]; [e-mail]; Francesca Palombini <[e-mail]>; RFC Errata System <[e-mail]>
Subject: Re: [stir] [Technical Errata Reported] RFC8224 (6519)

Hi Roman,

On 4/7/21 9:18 AM, Roman Shpount wrote:
> Hi Christer,
> 
> This is exactly the issue. ATIS documents and other RFCs like rfc8946 
> use ppt with a quoted token.

Shouldn't the ABNF now be, to be compatible with non-SHAKEN implementations, this:

ident-type = "ppt" EQUAL (token / quoted-string)

> 
> Also, according to IETF 101 STIR notes (which you took), the ppt token 
> value should always be quoted. I am not sure why it needed to be 
> quoted (I think this is wrong), but that was the decision.
> _____________
> Roman Shpount
> 
> 
> On Wed, Apr 7, 2021 at 11:23 AM Christer Holmberg < 
> [e-mail]> wrote:
> 
>> Hi,
>>
>> I have not been involved in the discussions, so my apologies for 
>> asking something that have been discussed already, but what is the 
>> reason for the change? If you anyway are only going to allow "token" 
>> characters, why the quotes? Is the only reason to align with how the 
>> parameter is used in other specs?
>>
>> Regards,
>>
>> Christer
>>
>>
>> -----Original Message-----
>> From: stir <[e-mail]> On Behalf Of RFC Errata System
>> Sent: tiistai 6. huhtikuuta 2021 8.21
>> To: [e-mail]; [e-mail]; [e-mail]; 
>> [e-mail]; [e-mail]; Francesca Palombini < 
>> [e-mail]>; [e-mail]; 
>> [e-mail]
>> Cc: [e-mail]; [e-mail]; [e-mail]
>> Subject: [stir] [Technical Errata Reported] RFC8224 (6519)
>>
>> The following errata report has been submitted for RFC8224, 
>> "Authenticated Identity Management in the Session Initiation Protocol (SIP)".
>>
>> --------------------------------------
>> You may review the report below and at:
>>
>> https://protect2.fireeye.com/v1/url?k=ac0542ba-f39e7bbf-ac050221-8695
>> 9e472243-713aff0f88c18be4&q=1&e=6be2f688-f156-4c8b-953f-2bec6cf24d76&
>> u=https%3A%2F%2Fwww.rfc-editor.org%2Ferrata%2Feid6519
>>
>> --------------------------------------
>> Type: Technical
>> Reported by: Roman Shpount <[e-mail]>
>>
>> Section: 4
>>
>> Original Text
>> -------------
>> ident-type = "ppt" EQUAL token
>>
>> Corrected Text
>> --------------
>> ident-type = "ppt" EQUAL DQUOTE token DQUOTE
>>
>> Notes
>> -----
>> Based on IETF 101 STIR notes ptr= values should always be quoted. 
>> Also,
>> ATIS-1000074 is using double quotes around ppt value.
>>
>> Instructions:
>> -------------
>> This erratum is currently posted as "Reported". If necessary, please 
>> use "Reply All" to discuss whether it should be verified or rejected. 
>> When a decision is reached, the verifying party can log in to change 
>> the status and edit the report, if necessary.
>>
>> --------------------------------------
>> RFC8224 (draft-ietf-stir-rfc4474bis-16)
>> --------------------------------------
>> Title               : Authenticated Identity Management in the Session
>> Initiation Protocol (SIP)
>> Publication Date    : February 2018
>> Author(s)           : J. Peterson, C. Jennings, E. Rescorla, C. Wendt
>> Category            : PROPOSED STANDARD
>> Source              : Secure Telephone Identity Revisited
>> Area                : Applications and Real-Time
>> Stream              : IETF
>> Verifying Party     : IESG
>>
>> _______________________________________________
>> stir mailing list
>> [e-mail]
>> https://www.ietf.org/mailman/listinfo/stir
>>
> 
> 
> _______________________________________________
> stir mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/stir
> 


--
Marc Petit-Huguenin
Email: [e-mail]
Blog: https://protect2.fireeye.com/v1/url?k=d873d4d4-87e8ed90-d873944f-861d41abace8-1ba57b7d439ca660&q=1&e=42151d84-9aa4-4d77-a4d9-6384ca516e8d&u=https%3A%2F%2Fmarc.petit-huguenin.org%2F
Profile: https://www.linkedin.com/in/petithug