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 Content-Type at /logout endpoint #218

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Fix Content-Type at /logout endpoint #218

merged 1 commit into from
Sep 19, 2024

Conversation

paskal
Copy link
Collaborator

@paskal paskal commented Sep 19, 2024

No text is returned so the response is not a valid JSON.

Fix for header introduced in #203, resolves umputun/remark42#1812. I verified the fix locally.

@paskal paskal requested a review from umputun as a code owner September 19, 2024 23:19
@coveralls
Copy link

coveralls commented Sep 19, 2024

Pull Request Test Coverage Report for Build 10950419206

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.08%) to 83.36%

Totals Coverage Status
Change from base Build 10642899118: -0.08%
Covered Lines: 2600
Relevant Lines: 3119

💛 - Coveralls

No text is returned so the response is not a valid JSON.
@paskal paskal changed the title Change Content-Type to plain text for logout and reset endpoints Change Content-Type to plain text for logout endpoint Sep 19, 2024
@paskal paskal changed the title Change Content-Type to plain text for logout endpoint Fix Content-Type at /logout endpoint Sep 19, 2024
@umputun umputun merged commit 458e57e into master Sep 19, 2024
9 checks passed
@umputun umputun deleted the paskal/plain_text branch September 19, 2024 23:45
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.

Encounter proplem in logging out
3 participants