-
Notifications
You must be signed in to change notification settings - Fork 16
Add "content" parameter to MIME type application/x-votable+xml #76
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
base: master
Are you sure you want to change the base?
Conversation
The content parameter has been used for some while by DataLink (e.g. application/x-votable+xml; content=datalink), so that parameter should really be defined in the standard where the MIME type itself is defined, i.e. VOTable. Reorganise the MIME type description a little bit and take the opportunity to update a couple of the referenced RFCs with ones by which they are obsoleted. Modernise advice about character encoding in line with RFC7303: advise always to use UTF-8, deprecate use of the charset parameter, and withdraw comments about a US-ASCII default which are no longer true.
msdemlei
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As a matter of terminological tidyness, I would probably be happier if we used the opportunity to change "MIME type" to "media type", which the IETF is recommending for all I can see; it also makes sense because the E in MIME is E-Mail, which is a bit remote from what we're doing here.
But then I don't care enough to do it myself, and so I'm not unhappy if VOTable keeps talking about MIME:
| \item {\textsf{text/xml}} | ||
| \item {\textsf{text/xml; charset={"}iso-8859-1"}} | ||
| \item {\textsf{application/x-votable+xml}} | ||
| \item {\textsf{APPLICATION/X-VOTABLE+XML; CONTENT=datalink}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you change the case here on purpose just to alert people to the media type's unfortunate case policy?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't think it was worth spelling out the case-insensitivity explicitly - I don't want to replicate the syntax rules from RFC 2045 here - but since we have examples here anyway I thought I'd take the opportunity to give an idea of the range of possible representations.
|
Changing MIME type to Media Type sounds reasonable - I'll do it if nobody expresses a contrary opinion. |
The content parameter has been used for some while by DataLink (e.g. application/x-votable+xml; content=datalink), so that parameter should really be defined in the standard where the MIME type itself is defined, i.e. VOTable.
Reorganise the MIME type description a little bit and take the opportunity to update a couple of the referenced RFCs with ones by which they are obsoleted.
Modernise advice about character encoding in line with RFC7303: advise always to use UTF-8, deprecate use of the charset parameter, and withdraw comments about a US-ASCII default which are no longer true.