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 a commit
that referenced
this issue
Oct 26, 2018
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
referenced this issue
Mar 17, 2019
Open
Consider HTTP 300 as realization for list profiles / list tokens #549
larsgsvensson
added a commit
to ProfileNegotiation/I-D-Profile-Negotiation
that referenced
this issue
Mar 30, 2019
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
? |
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