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

'File' type validation #1

Closed
webron opened this issue Apr 24, 2014 · 1 comment
Closed

'File' type validation #1

webron opened this issue Apr 24, 2014 · 1 comment

Comments

@webron
Copy link
Contributor

webron commented Apr 24, 2014

According to - https://github.com/wordnik/swagger-spec/blob/master/versions/1.2.md#434-file - "When using File, the consumes field MUST be "multipart/form-data", and the paramType MUST be "form". ".

In the pet store sample there's a bug where the paramType is "body" - http://petstore.swagger.wordnik.com/api/api-docs/pet - check out the POST operation on "/pet/uploadImage".
fge added a commit that referenced this issue Apr 24, 2014
@fge
Reproduce issue #1
dc298e3
fge added a commit that referenced this issue Apr 24, 2014
@fge
Fix issue #1: modify parameter object schema
c12a508 
Use oneOf:

* either the object must not have "File" as a "type";
* or its "paramType" and "consumes" members must comply.
@webron webron mentioned this issue Apr 28, 2014
Closed
@webron webron unassigned fge Oct 20, 2014
@webron
Copy link
Contributor Author

webron commented Feb 13, 2015

Closing for now, will reopen should an issue arise.

@webron webron closed this as completed Feb 13, 2015
@fehguy fehguy mentioned this issue Mar 7, 2016
Closed
fehguy pushed a commit that referenced this issue Mar 14, 2016
@demonfiddler
Merge pull request #1 from swagger-api/master
9561f9d 
Pull remote changes
fehguy pushed a commit that referenced this issue Jun 7, 2017
@msymons
Merge pull request #1 from msymons/msymons-patch-1
6b81a76 
Update json-schema-validator (SONATYPE-2015-0090)
fehguy pushed a commit that referenced this issue Jun 12, 2017
@gracekarina
Merge pull request #1 from fehguy/feature/3.0.0-rc0
1e4cdf4 
Updated to 3.0.0-rc1
cvgaviao pushed a commit to cvgaviao/swagger-parser that referenced this issue Apr 9, 2019
@wing328
Merge pull request swagger-api#1 from OpenAPITools/fix-oas2-parameter…
80ace3f 
…-conversion

Fix OAS2 parameter conversion
gracekarina pushed a commit that referenced this issue May 15, 2019
@kerrykimbrough
Merge pull request #1 from swagger-api/master
8af23f4 
Update to latest swagger-api/swagger-parser
gracekarina pushed a commit that referenced this issue Aug 10, 2019
@scottr-ad
Merge pull request #1 from swagger-api/master
59a1d1a 
Merge pull request #1134 from scottr-ad/master
gracekarina pushed a commit that referenced this issue Oct 9, 2019
@r-sreesaran
Merge pull request #1 from swagger-api/master
f8c6f35 
latest changes
gracekarina pushed a commit that referenced this issue Oct 19, 2022
@mgiamberardino
Merge pull request #1 from swagger-api/master
e059e0c 
Get latest changes from swagger-parser origin repository
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@webron @fge