Skip to content

fix: add UTF-8 charset to OAuth callback HTML responses#8

Merged
scottlovegrove merged 1 commit intomainfrom
frankieyan/oauth-fix
Jan 22, 2026
Merged

fix: add UTF-8 charset to OAuth callback HTML responses#8
scottlovegrove merged 1 commit intomainfrom
frankieyan/oauth-fix

Conversation

@frankieyan
Copy link
Copy Markdown
Member

@frankieyan frankieyan commented Jan 22, 2026

Fixed character encoding issue in OAuth callback pages where checkmark (✅) displayed as mojibake

Similar fix as Doist/todoist-cli#17

Before After
image image

Test plan

  • Run tw auth login
  • Verify success page displays checkmark (✅) correctly in browser

The checkmark (✅) and cross (❌) characters in the auth success/error
pages displayed as mojibake in browsers due to missing charset
declaration. Add charset=utf-8 to Content-Type headers and meta charset
tags to HTML.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@scottlovegrove
Copy link
Copy Markdown
Collaborator

You know, I just figured that was just some weird "we show foreign letters" thing 😄

@scottlovegrove scottlovegrove merged commit fcb2d49 into main Jan 22, 2026
2 checks passed
@scottlovegrove scottlovegrove deleted the frankieyan/oauth-fix branch January 22, 2026 17:36
github-actions Bot pushed a commit that referenced this pull request Jan 22, 2026
# 1.0.0 (2026-01-22)

### Bug Fixes

* add UTF-8 charset to OAuth callback HTML responses ([#8](#8)) ([fcb2d49](fcb2d49))

### Features

* Add Biome linting, upgrade to Node 20, and add CI workflows ([#3](#3)) ([57f802e](57f802e))
* add loading animations with global API proxy integration ([#7](#7)) ([bf53bfb](bf53bfb))
* Add OAuth authentication with dynamic client registration ([#6](#6)) ([2845ebd](2845ebd)), closes [#5](#5)
* add semantic release automation for NPM publishing ([#10](#10)) ([c756324](c756324))
* Refactor login command to auth with status/logout subcommands ([#4](#4)) ([59e6493](59e6493))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants