Skip to content
This repository has been archived by the owner on Jul 2, 2019. It is now read-only.

Default response is omitted #13

Closed
freaz opened this issue Nov 25, 2015 · 4 comments
Closed

Default response is omitted #13

freaz opened this issue Nov 25, 2015 · 4 comments

Comments

@freaz
Copy link
Contributor

freaz commented Nov 25, 2015

https://github.com/apiaryio/swagger-zoo/blob/master/fixtures/features/swagger/default-response.yaml

@pksunkara
Copy link
Contributor

This was supposed to be intentional if I remember. @danielgtaylor or @smizell would know more.

@danielgtaylor
Copy link
Contributor

We deliberately ignore the default response because I'm not sure how it should be represented. In Swagger you would return it for any type of response that isn't explicitly listed but in Refract we expect every possible response to be listed explicitly. We need to figure out what to do with this.

@erajkovic
Copy link

erajkovic commented May 12, 2018

If you are talking about this warning with the fury tool:
warning: (3) Default response is not yet supported
Then yes, we need to figure a way to support this as it's becoming a common pattern when you have larger Swagger documents.
The main benefit I see, as a Swagger author is that I do not have to cover '500', '502', '503', and a bunch of 3xx / 4xx error codes as well - They are mostly related to the proxy layers before to reach my application code.
Sample Swagger files:
https://apicatalog.oraclecloud.com/public/v1/orgs/oracle-public/apicollections/apicatalog/1.0/apis/API%20Collections/canonical
https://apicatalog.oraclecloud.com/public/v1/orgs/oracle-public/apicollections/commerce/17.4/apis/admin/canonical

@kylef
Copy link
Member

kylef commented Jul 2, 2019

Moved to apiaryio/api-elements.js#264

@kylef kylef closed this as completed Jul 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants