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

Set content-type and charset #26

Merged
merged 3 commits into from
Jun 19, 2023

Conversation

biryukovmaxim
Copy link
Contributor

set content type header.
it's better for client to decode response, it's good also for debugging using sniffers

@Totodore
Copy link
Owner

Did you consider adding content types on other responses (for instance in error responses) ?

@biryukovmaxim
Copy link
Contributor Author

biryukovmaxim commented Jun 19, 2023

Did you consider adding content types on other responses (for instance in error responses) ?

I thought I did it in the error handling pr. I checked and realized that I didn't.

@Totodore Totodore merged commit 0bd8881 into Totodore:main Jun 19, 2023
4 checks passed
@biryukovmaxim biryukovmaxim deleted the default-content-type branch June 26, 2023 09:47
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