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

[wip] fix: content-type header for metrics endpoints #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vakhov
Copy link

@vakhov vakhov commented Dec 28, 2024

Issue #13:

  • JSON endpoints now return the correct Content-Type: application/json; charset=utf-8.

Additionally:

  • Added tests to verify the correctness of the Content-Type header.

This fix ensures proper header formatting and alignment with client expectations.

@vakhov vakhov force-pushed the fix/content-type-header branch from fa3e31b to 203ddf5 Compare December 28, 2024 11:31
@vakhov vakhov changed the title Fix Content-Type header for metrics endpoints fix: Content-Type header for metrics endpoints Dec 28, 2024
@vakhov vakhov force-pushed the fix/content-type-header branch from 203ddf5 to f788f0c Compare December 28, 2024 11:44
@vakhov vakhov changed the title fix: Content-Type header for metrics endpoints [wip] fix: content-type header for metrics endpoints Dec 28, 2024
@vakhov vakhov force-pushed the fix/content-type-header branch 2 times, most recently from 814fe8b to f18401a Compare December 28, 2024 16:40
    Issue tarantool#13:
    - JSON endpoints now return the correct Content-Type: `application/json; charset=utf-8`.

    Additionally:
    - Added tests to verify the correctness of the Content-Type header.

    This fix ensures proper header formatting and alignment with client expectations.
@vakhov vakhov force-pushed the fix/content-type-header branch from f18401a to 199cc67 Compare December 28, 2024 17:13
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.

1 participant