-
Notifications
You must be signed in to change notification settings - Fork 3
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
Response Content Type in Swagger #21
Comments
Can you give an example? Apparently a problem like that just crashed the server, would like to reproduce. |
The bug happens for |
See #20 the example |
There was some confusion of media types across the rdf processing libraries. Should be fixed now. |
Yes, now it works to choose a non-standard option in the response content type withouth a crash and the The |
It's a bug in Swagger. Updated the library to the latet version and it's the same error. It doesn't work for the bundled HTML app either (try: http://infolis.gesis.org/infolink/vendor/swagger-ui/dist/index.html). Not much we can do short of digging into the Swagger JS (the The important thing is that the requests are sent to the server correctly. |
It doesn't work on the official demo site either: http://petstore.swagger.io/ with http://infolis.gesis.org/infolink/swagger.json |
It is confirmed that this is a swagger-ui bug and it was given the tag |
Seems fixed upstream by swagger-api/swagger-ui@8276fbf |
Yes, swagger-api/swagger-ui#1767 (comment) is fixed and so is this. |
Yes they should 😒 |
Is that standard field in swagger-ui? I think I haven't seen this in the demo example from swagger-ui itself... |
Yes, it's a standard field, but it's disabled in the petstore demo. I disabled it here as well. |
Using a non-standard option in the response content type (e.g. +
rdf+xml
) does not affect thecurl
action shown below or therequest headers
.The text was updated successfully, but these errors were encountered: