This is a proposal to address a number of the issues aggregated under #560.
Specifically:
The basic premise is that OpenAPI models operations that interact with resources using a specific HTTP method. The interaction involves sending representations and retrieving those representations. A single operation may support different representations identified by a media-type.
Consider the following GET operation that returns one of two representations.
get:
description: Returns pets based on ID
summary: Find pets by ID
operationId: getPetsById
responses:
'200':
description: pet response
representations:
application/json:
schema:
type: array
items:
$ref: '#/definitions/Pet'
text/html:
default:
description: error payload
representations:
application/json:
schema:
$ref: '#/definitions/ErrorModel'
text/html:
parameters:
- name: id
in: path
description: ID of pet to use
required: true
type: array
items:
type: string
collectionFormat: csv
The description and headers properties of a response object stay, but the schema and examples move under the representation object that is defined as a property of the representations object. The examples property will need to change to either an array of objects, or a single object because all examples would be for the same media type.
By defining the supported media types on the representations object, there is no longer a need for a produces array.
It is important to note that different representations should not be semantically different when accompanied with the same class of status code. A request to a resource should always the same thing (for some unfortunately nebulous definition of thing). However, the syntax of that representation may be different and the amount of information contained may be different, but from a consumer's perspective. it is same concept, regardless of the representation. This is why the description property is the same for all representations.
The following is an example of a POST request that may send a HTML form as a request body.
tags:
- pet
summary: Updates a pet in the store with form data
description: ""
operationId: updatePetWithForm
parameters:
- name: petId
in: path
description: ID of pet that needs to be updated
required: true
type: string
requestbody:
description: Updated status of the pet
required: false
representations:
application/x-www-form-urlencoded:
schema:
properties:
name:
description: Updated name of the pet
type: string
status:
description: Updated status of the pet
type: string
required:
- status
responses:
'200':
description: Pet updated.
representations:
application/json:
application/xml:
'405':
description: Invalid input
representations:
application/json:
application/xml:
security:
- petstore_auth:
- write:pets
- read:pets
The structure of the HTML form that is passed as a body are no longer intermixed with the URI parameters and are described by a schema object in the representation object. This enables us to support all the different form related media types. There would no longer be any need for the formData parameter type and no need for the consumes array.
One open question is whether there is a value to allowing representation objects (media type, schema and examples) to be defined within the reusable components section.
I'm not sure that your text/html part is valid YAML. (I guess this is supposed to be an empty object? Then it should be text/html: {}.)
@ePaul I wasn't sure about that, but I ran it through YAMLLint and it validated ok.
And is it parsed as an empty object or an empty array? Or a null?
Yay +1.
Just ran this through with YAMLLint using a format for the content type that includes the profile. It validated it as valid YAML, but only after it reformatted it a bit like so:
requestbody:
description: "Updated status of the pet"
representations:
? "application/json; profile=\"vnd.example.resource.v1\""
:
schema:
properties:
name:
description: "Updated name of the pet"
type: string
status:
description: "Updated status of the pet"
type: string
required:
- status
required: false
I'm not familiar enough with obscure (to me) YAML syntax to know whether it's appropriately treating that as a key for that value. Can someone with more knowledge confirm, please?
@vrrobz The ? is a way of being explicit about something being a key. I don't think there is any ambiguity in our case. http://yaml.org/spec/1.2/spec.html#id2798057
Awesome, that's what I needed to know. Thanks!
What is the JSON equivalent of yaml ? : notation? (We should be careful to not use YAML specific constructs)
The JSON equivalent to the first example is:
{
"get": {
"description": "Returns pets based on ID",
"summary": "Find pets by ID",
"operationId": "getPetsById",
"responses": {
"200": {
"description": "pet response",
"representations" : {
"application/json" : {
"schema": {
"type": "array",
"items": {
"$ref": "#/definitions/Pet"
}
}
},
"text/html" : {}
}
},
"default": {
"description": "error payload",
"representations" : {
"application/json" : {
"schema": {
"$ref": "#/definitions/ErrorModel"
}
},
"text/html" : {}
}
}
}
},
"parameters": [
{
"name": "id",
"in": "path",
"description": "ID of pet to use",
"required": true,
"type": "array",
"items": {
"type": "string"
},
"collectionFormat": "csv"
}
]
}
To my knowledge, I didn't use anything that can't be done in JSON. The : operator in Yaml is just a map, which maps to a JSON object.
For representations in application/xml I'd like to be able to reference an XML Schema.
@ralfhandl I agree. Supporting different schemas like XML Schema, Relax NG, Schematron, and alternative JSON validation like JCR seems like a ideal goal to me.
The challenge is that we would need to find a way to decouple the OpenAPI description mechanisms from the schema validation process. The benefit is that OpenAPI tooling would not have to reinvent schema validation mechanisms. But, tooling would have to deal with the fact that it might run into a validation mechanism that it doesn't know how to process.
In #333 a schemaType or schemaLanguage property was proposed to describe the type/language used in the schema property. The default/fallback could be the OpenAPI subset of JSON Schema.
That way tools can skip schemas whose language they can't cope with.
representing the schema for non-JSON structures is something we'll tackle outside of this PR. but I do think that this fixes the association between the content format and consumes and produces, by coupling them together. So +1 to this.
Will turn this into a PR for both the input (requestBody) parameter, and for response representations. We will keep the parameter serialization outside of the scope of the PR.
consider having examples: [] as a sibling to schema in the representation object.
This has been implemented in 3.0.0-RC0! 馃帀
Most helpful comment
In #333 a
schemaTypeorschemaLanguageproperty was proposed to describe the type/language used in theschemaproperty. The default/fallback could be the OpenAPI subset of JSON Schema.That way tools can skip schemas whose language they can't cope with.