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.

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

Fixed Accept header charset extra quotes bug which doesn't work on some ... #720

Closed
wants to merge 1 commit into from

Conversation

shifu
Copy link

@shifu shifu commented Nov 13, 2014

...servers

@fehguy
Copy link
Contributor

fehguy commented Dec 3, 2014

This will break servers that don't like the quotes, and quotes are not part of the RFC. I recommend joining in this issue:

#708 (comment)

And helping make the content-type headers configurable when initializing swagger-ui. Please reopen if I'm missing something

@fehguy fehguy closed this Dec 3, 2014
@shifu
Copy link
Author

shifu commented Dec 4, 2014

"This will break servers that don't like the quotes, and quotes are not part of the RFC." This is exactly what I am trying to fix. I am removing the quotes not adding them.

The extra quotes right now are not part of RFC and is breaking some servers. Am I missing something?

@shifu
Copy link
Author

shifu commented Dec 4, 2014

@fehguy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants