Re: [Gen-art] Genart last call review of draft-ietf-httpapi-link-template-02

2023-05-19 Thread Christer Holmberg
Hi Mark,

 Q2_2: The text says "Parameter values MUST be Strings."

 It is unclear what "Strings" means. Does it mean that parameter
 values must be encoded as quoted-strings? If so, why? RFC8288 says
 that parameter values can be encoded both as token and quoted-string.
>>>
>>> S 1.1: "This specification uses the following terms from
>>> [STRUCTURED-FIELDS]: List, String, Parameter."
>>
>> Yes. But, in Section 3.1.2 of [STRUCTURED-FIELDS] there is no
>> restriction that the Parameter value must by String. My question is
>> why you are making that restriction, instead of just allowing the
>> different Parameter value encodings defined in [STRUCTURED-FIELDS]?
>
> Ah. The reason is that allowing any type would require creating a mapping of 
> current values to SF types, and there
> are just too many potential (and undocumented) values already in use to do 
> this.

I don't think that is true. Just because the Parameter syntax allows values to 
be encoded sf-string, sf-token, sf-boolean etc it doesn't mean that you have 
to map each value (existing or new ones) to each of those encodings. If a 
value is defined as a String, then it has to be encoded as a sf-string.

Having said that, I'm fine with having the restriction, because in reality I 
assume the values always will have to be encoded as sf-strings anyway (you 
can't encode a URI as a sf-integer, sf-boolean etc).

Thanks!

Regards,

Christer




smime.p7s
Description: S/MIME cryptographic signature
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art


Re: [Gen-art] Genart last call review of draft-ietf-httpapi-link-template-02

2023-05-19 Thread Mark Nottingham
Hi Christer,

Thanks, see responses below.

> On 19 May 2023, at 1:13 pm, Christer Holmberg 
>  wrote:
> 
> Hi Mark,
> 
> Please see inline.
> 
>>> Q2_1: There is no ABNF for the header field. There are examples using
>>> both quotes ("/{username}") and angle brackets
>>> (), so please include the ABNF.
>> 
>> This is a Structured Field; in the HTTP community, we've agreed that 
>> documenting them with ABNF is not good practice.
> 
> Gotha.
> 
> (I was comparing this to RFC 8288, and assumed that you still would have to 
> define the ABNF, following the rules of [STRUCTURED-FIELDS])
> 
> But, then I have the following question:
> 
> Section 2 says:   "Its value is a List of Strings.  Each String is a URI 
> Template..."
> 
> According to [STRUCTURED-FIELDS]), the syntax for String is:  sf-string = 
> DQUOTE *chr DQUOTE
> 
> From the example in Section 2,  does not follow that 
> syntax, does it? You must use DQUOTEs.

Yes, that was a transposition error that's since been corrected.


>>> Q2_2: The text says "Parameter values MUST be Strings."
>>> 
>>> It is unclear what "Strings" means. Does it mean that parameter values
>>> must be encoded as quoted-strings? If so, why? RFC8288 says that
>>> parameter values can be encoded both as token and quoted-string.
>> 
>> S 1.1: "This specification uses the following terms from 
>> [STRUCTURED-FIELDS]: List, String, Parameter."
> 
> Yes. But, in Section 3.1.2 of [STRUCTURED-FIELDS] there is no restriction 
> that 
> the Parameter value must by String. My question is why you are making that 
> restriction, instead of just allowing the different Parameter value encodings 
> defined in [STRUCTURED-FIELDS]?

Ah. The reason is that allowing any type would require creating a mapping of 
current values to SF types, and there are just too many potential (and 
undocumented) values already in use to do this. 

Cheers,

--
Mark Nottingham   https://www.mnot.net/

___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art


Re: [Gen-art] Genart last call review of draft-ietf-httpapi-link-template-02

2023-05-19 Thread Christer Holmberg
Hi Mark,

Please see inline.

>> Q2_1: There is no ABNF for the header field. There are examples using
>> both quotes ("/{username}") and angle brackets
>> (), so please include the ABNF.
>
> This is a Structured Field; in the HTTP community, we've agreed that 
> documenting them with ABNF is not good practice.

Gotha.

(I was comparing this to RFC 8288, and assumed that you still would have to 
define the ABNF, following the rules of [STRUCTURED-FIELDS])

But, then I have the following question:

Section 2 says:   "Its value is a List of Strings.  Each String is a URI 
Template..."

According to [STRUCTURED-FIELDS]), the syntax for String is:  sf-string = 
DQUOTE *chr DQUOTE

>From the example in Section 2,  does not follow that 
syntax, does it? You must use DQUOTEs.



>> Q2_2: The text says "Parameter values MUST be Strings."
>>
>> It is unclear what "Strings" means. Does it mean that parameter values
>> must be encoded as quoted-strings? If so, why? RFC8288 says that
>> parameter values can be encoded both as token and quoted-string.
>
> S 1.1: "This specification uses the following terms from 
> [STRUCTURED-FIELDS]: List, String, Parameter."

Yes. But, in Section 3.1.2 of [STRUCTURED-FIELDS] there is no restriction that 
the Parameter value must by String. My question is why you are making that 
restriction, instead of just allowing the different Parameter value encodings 
defined in [STRUCTURED-FIELDS]?

Regards,

Christer



smime.p7s
Description: S/MIME cryptographic signature
___
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art