Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

one more reason for defining a profile parameter #7

Closed
thomas-fossati opened this issue Jul 18, 2022 · 1 comment · Fixed by #11
Closed

one more reason for defining a profile parameter #7

thomas-fossati opened this issue Jul 18, 2022 · 1 comment · Fixed by #11
Milestone

Comments

@thomas-fossati
Copy link
Collaborator

Dave Thaler in https://mailarchive.ietf.org/arch/msg/rats/5BZdOsNNquo9l6nuUSEbkobOWiY/

I have just one wording suggestion.  In section 3 it says:

> The media types defined in this document include an optional profile
> parameter that can be used to mirror the eat_profile claim of the
> transported EAT.  Exposing the EAT profile at the API layer allows
> API routers to dispatch payloads directly to the profile-specific
> processor without having to snoop into the request bodies. 
 
That is of course true, but the justification can be even stronger by pointing out that it also allows use when the payload is not present in the message, such as in an Accept header (and indeed section 4 shows that).  
 
Indeed, the TEEP protocol uses it in both Content-Type and Accept, just like in section 4.
@thomas-fossati
Copy link
Collaborator Author

thomas-fossati commented Jul 19, 2022

Also, @carl-wallace on the same thread:

It may be worth adding an informational reference to RFC4151 for the tag URIs in the
examples. Maybe add this to the sentence at the end of section 4:
 
“In both cases, a tag URI [RFC4151] identifying the profile is carried as an explicit
parameter.”

thomas-fossati added a commit that referenced this issue Oct 16, 2022
* DEB (the acronym) is no more
* add refs to REST design documents for HTTP and CoAP
* two edits suggested by Carl and Dave

Fix #7
Fix #8
Fix #9

Signed-off-by: Thomas Fossati <thomas.fossati@arm.com>
@thomas-fossati thomas-fossati added this to the -01 milestone Oct 16, 2022
thomas-fossati added a commit that referenced this issue Oct 17, 2022
* rename the media-type parameter from `profile` to `eat_profile`
* DEB (the acronym) is no more
* add refs to REST design documents for HTTP and CoAP
* two edits suggested by Carl and Dave

also:

* add line RFC8792 line folding in HTTP examples
* add a changelog

Fix #4
Fix #7
Fix #8
Fix #9

Signed-off-by: Thomas Fossati <thomas.fossati@arm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant