Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upRegistration of target attribute "profile" for the Link-Header #501
Comments
larsgsvensson
added
the
profile-negotiation
label
Oct 26, 2018
This comment has been minimized.
This comment has been minimized.
I don’t understand as I see it register here: |
This comment has been minimized.
This comment has been minimized.
https://www.iana.org/assignments/link-relations/link-relations.xhtml#link-relations-1 is the registry of link relation types (i. e. what comes in |
This comment has been minimized.
This comment has been minimized.
profile is defined - and explained here https://tools.ietf.org/html/rfc6906 AFIACT it is consistent with our model, but not as explicitly defined (seems a bit vague really) note that is says: |
This comment has been minimized.
This comment has been minimized.
I might have gotten this wrong, so apologies up front... |
This comment has been minimized.
This comment has been minimized.
ahh - i see - you dont want the link relation "profile" at all - we need to explain this better both both cases. so rel="profile" is to designate the profile of the current resource Agreed - we can see what about the list of alternates (profiles) supported - does this need a special link rel too? |
This comment has been minimized.
This comment has been minimized.
@rob-metalinkage scripsit:
My suggestion is to have a |
This comment has been minimized.
This comment has been minimized.
azaroth42
commented
Nov 8, 2018
|
larsgsvensson
added this to To do
in Content Negotiation by Profile
Nov 14, 2018
This comment has been minimized.
This comment has been minimized.
+1 to @azaroth42: the WD as just published seems to assume that there is a registry of target attributes. there is no such thing. RFC 5988 just was updated (see #632), and there still is no |
This comment has been minimized.
This comment has been minimized.
Just as a reminder to myself: The syntax for the profile attribute must somehow also allow the server to map a profile URI to a profile token. Cf. #290 |
larsgsvensson
moved this from To do
to In progress
in Content Negotiation by Profile
Apr 3, 2019
larsgsvensson
self-assigned this
Apr 3, 2019
This comment has been minimized.
This comment has been minimized.
There is now text in the proposed I-D that defines the link header target attribute "profile" |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
On 2019-06-19 06:01, Lars G. Svensson wrote:
See also @hvdsomp <https://github.com/hvdsomp> 's suggestion to include
the attribute 'profile' in the new Link-Hints registry
<ProfileNegotiation/I-D-Profile-Negotiation#17 (comment)>
just as a note: the link-hint draft has been around for a while and
abandoned for a while, and it's less than certain that the draft (and
thus the registry) will make it to the finish line.
|
This comment has been minimized.
This comment has been minimized.
hvdsomp
commented
Jun 20, 2019
Yes, I get that. I still feel that this attribute should be defined generically, not tied to this specific spec. Solidarity!
… On Jun 20, 2019, at 18:06, Erik Wilde ***@***.***> wrote:
On 2019-06-19 06:01, Lars G. Svensson wrote:
> See also @hvdsomp <https://github.com/hvdsomp> 's suggestion to include
> the attribute 'profile' in the new Link-Hints registry
> <ProfileNegotiation/I-D-Profile-Negotiation#17 (comment)>
just as a note: the link-hint draft has been around for a while and
abandoned for a while, and it's less than certain that the draft (and
thus the registry) will make it to the finish line.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
This comment has been minimized.
This comment has been minimized.
@hvdsomp scripsit:
we're certainly on the same page when it comes to solidarity. And I don't see how defining the 'profile' attribute in the RFC restricts it to be used only within the scope of that specific spec. |
This comment has been minimized.
This comment has been minimized.
I have a general feeling that this issue has been adequately discussed and that we've reached consensus that the text in the IETF draft covers this. @dret, @hvdsomp, @azaroth42: do you agree? |
This comment has been minimized.
This comment has been minimized.
On 2019-08-19 11:51, Lars G. Svensson wrote:
I have a general feeling that this issue has been adequately discussed
and that we've reached consensus that the text in the IETF draft
<https://profilenegotiation.github.io/I-D-Profile-Negotiation/I-D-Profile-Negotiation.html#rfc.section.6>
covers this. @dret <https://github.com/dret>, @hvdsomp
<https://github.com/hvdsomp>, @azaroth42 <https://github.com/azaroth42>:
do you agree?
If yes, I think we can mark this issue as "due-for-closing"
fine with me. thanks!
|
This comment has been minimized.
This comment has been minimized.
hvdsomp
commented
Aug 19, 2019
Lacking a registry for target attributes, defining one in an RFC is probably the next best thing. It can be interpreted to be aligned with the Target Attributes section of RFC8288. I wonder, however, whether the text in the current I-D makes the below intent expressed by @larsgsvensson clear enough, especially since that text also talks about Content-Profile:
|
This comment has been minimized.
This comment has been minimized.
@hvdsomp scripsit:
Would it help to change this text to
? |
This comment has been minimized.
This comment has been minimized.
azaroth42
commented
Aug 27, 2019
Yes, fine by me, thank you! |
This comment has been minimized.
This comment has been minimized.
hvdsomp
commented
Aug 28, 2019
Oops, I had missed @larsgsvensson above suggestion. Minimalist change but ok for me. Cheers. |
This comment has been minimized.
This comment has been minimized.
OK, thank you all. There seems to be consensus that the text in the IETF draft covers this:
Now we only have to resolve #290 and we're ready to close. |
This comment has been minimized.
This comment has been minimized.
The discussion over in #290 (unfortunately) has implications for this issue, too.
Currently, the I-D defines the value of the
My personal feeling is that defining an extension to the
As |
This comment has been minimized.
This comment has been minimized.
@larsgsvensson Agree with your assessment; extending the syntax is to be avoided. |
This comment has been minimized.
This comment has been minimized.
@larsgsvensson I think that is a good way to resolve this - if the IETF allows for additional parameters other specifications can standardise what they mean. one minor tweak would be to allow token values to be lists (separated by , ? ) Also - it may be useful to allow servers to indicate what they know about profile hierarchies in a special token ( eg profileof=A,B,C) - if we can use URIs as token values legally? |
This comment has been minimized.
This comment has been minimized.
@RubenVerborgh scripsit:
Does that mean that you agree that we extend the syntax for the |
This comment has been minimized.
This comment has been minimized.
hvdsomp
commented
Sep 2, 2019
I seem to remember having expressed significant concerns about the token approach in a private email exchange with @larsgsvensson and @RubenVerborgh. A very significant complication that is hard to motivate. |
This comment has been minimized.
This comment has been minimized.
@hvdsomp scripsit:
Yes, you have. That said, I think that an extensible syntax for the |
This comment has been minimized.
This comment has been minimized.
@rob-metalinkage scripsit:
RFC 7230 §3.2.6 defines
That means that URIs cannot be used as tokens since e. g. the colon character is not allowed in tokens. |
larsgsvensson commentedOct 26, 2018
•
edited
The examples given for the use of the http
Link
header uses the target attributeprofile
that is not yet registered. Cf. https://tools.ietf.org/html/rfc5988#section-5.4