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.

Sign up for GitHub

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

Jump to bottom

Definition of content type #49

Open
LPardue opened this issue Apr 20, 2022 · 3 comments
Open

Definition of content type #49

LPardue opened this issue Apr 20, 2022 · 3 comments
Assignees
@randall

Comments

@LPardue
Copy link
Contributor

LPardue commented Apr 20, 2022

Please consider defining/recommending/citing the media type of HTTP message content aka the value in the "Content-Type" header. Under HTTP rules, a sender SHOULD set that.

For instance, the JSON config could use the "application/json" type. While the upload / download resources could be "application/octet-stream".
cpaasch added a commit that referenced this issue Jul 6, 2022
@cpaasch
Definition of content type #49
70a7eeb 
Github issue #49
@cpaasch
Copy link
Contributor

cpaasch commented Jul 6, 2022

Added clarification on content-type.

@cpaasch cpaasch closed this as completed Jul 6, 2022
@LPardue
Copy link
Contributor Author

LPardue commented Jul 11, 2022

Thanks looks good.

I've just realised the text still omits the content-type of the upload, which I missed on the first read through and neglected to mention in the issue.

@cpaasch
Copy link
Contributor

cpaasch commented Jul 11, 2022

@randall - what were your thoughts on the upload content-type?

@cpaasch cpaasch reopened this Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@randall randall

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cpaasch @randall @LPardue