ietf
[Top] [All Lists]

Re: Last Call: <draft-nottingham-rfc5988bis-05.txt> (Web Linking) to Proposed Standard

2017-05-02 14:22:02
Review of draft-nottingham-rfc5988bis-05.txt
Reviewer: Carsten Bormann
Review result: Ready with a few issues

(This is not a complete review, but in its present state might serve
to initiate discussion of items relevant to RFC 6690 and thus
draft-ietf-core-links-json.)

This specification updates RFC 5988.  The objectives for this update
are not clearly stated, but it seems to be based on experience with
RFC 5988 as well as based on advances possible with the publication of
RFC 7230.

# Major technical

T1: The draft does not take position what should happen when
serializations allowed by RFC 5988 but not by the present spec occur,
e.g.: ;type=text/plain (would now need to be ;type="text/plain").

T2: The draft says "Serialisations SHOULD coordinate their target
attributes to avoid conflicts in semantics or syntax."  To me it seems
that this gives link applications such as the one specified in RFC
6690 the go-ahead to define their own registries of target attributes.
The draft should take a position on this.

# Minor technical

T3: One of the major items of progress that this specification exhibits
is that target attributes are no longer defined by the ABNF of the
link-header serialization (which usually has two alternatives, one of
which may be forgotten) but by the ABNF of the attribute value string
itself.  ANBF tools usually can process ABNF rules, but not directly
the bare ABNF "alternations" (rule RHS) used here.  This may also make
it a bit harder to reference the ABNF from a dependent specification,
as there is no rule name for that alternation given.

# Minor editorial

E1: The abstract should probably mention that this replaces RFC 5988.

# Nits

The spec seems to use 'NOT RECOMMENDED' as an RFC 2119 keyword, but
does not include the phrase in its RFC 2119 key words list.  (Pet
peeve.)

s/ for indicate/ for indicating/

Grüße, Carsten