search

json-schema-org / website

JSON Schema website
https://json-schema.org
Other
54 stars 141 forks source link

contentEncoding incorrectly includes binary #191

Closed spacether closed 6 months ago

spacether commented 2 years ago

contentEncoding incorrectly includes binary It should be updated to not include it https://json-schema.org/understanding-json-schema/reference/non_json_data.html

handrews commented 2 years ago

Technically it's correct that draft-07 included "binary". But later drafts do not, and it never made any sense anyway. RFC 2045 confusingly mixed the body domain (binary, 7bit, 8bit) and others perform transformations (quoted-printable, base64). Only the transformations are relevant - the binary domain in particular is not valid JSON, or allowable by the draft-07 data model.

(thanks for filing, @spacether)

benjagm commented 6 months ago

Closed as completed. Thanks everyone!