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

Fix "Services' status" page content type #1575

Merged
merged 3 commits into from
Mar 30, 2022

Conversation

colega
Copy link
Contributor

@colega colega commented Mar 30, 2022

What this PR does

It was forced to be text/plain for some reason. Removed that forcing, and added an explicit setting of text/html in RenderHTTPResponse when it's not YAML (comment says it will render HTML, so it's safe).

Which issue(s) this PR fixes or relates to

A new take to #1265

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

It was forced to be `text/plain` for some reason. Removed that forcing,
and added an explicit setting of `text/html` in RenderHTTPResponse when
it's not YAML (comment says it will render HTML, so it's safe).

Signed-off-by: Oleg Zaytsev <[email protected]>
@colega colega changed the title Fix "Service status" page content type Fix "Services' status" page content type Mar 30, 2022
Signed-off-by: Oleg Zaytsev <[email protected]>
Copy link
Member

@pstibrany pstibrany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

What is utf=8? Apparently my browser just ignored that.

Signed-off-by: Oleg Zaytsev <[email protected]>
@colega colega merged commit 75372f7 into main Mar 30, 2022
@colega colega deleted the fix-service-status-page-content-type branch March 30, 2022 16:44
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