[PR #21] [MERGED] Add '$encoding' parameter to getQRCodeInline() to avoid CJK issues. #344

Closed
opened 2026-03-01 17:49:20 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/antonioribeiro/google2fa/pull/21
Author: @zhiyi7
Created: 12/18/2015
Status: Merged
Merged: 5/9/2016
Merged by: @antonioribeiro

Base: masterHead: master


📝 Commits (1)

  • 6e8c7b4 Add '$encoding' parameter to getQRCodeInline() to avoid CJK issues.

📊 Changes

1 file changed (+6 additions, -1 deletions)

View changed files

📝 src/Google2FA.php (+6 -1)

📄 Description

If $company or $holder includes CJK(or any UTF8) character, the default encoding ISO8859-1 which used in BaconQrCodeGenerator will cause an error while it generating QR code.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/antonioribeiro/google2fa/pull/21 **Author:** [@zhiyi7](https://github.com/zhiyi7) **Created:** 12/18/2015 **Status:** ✅ Merged **Merged:** 5/9/2016 **Merged by:** [@antonioribeiro](https://github.com/antonioribeiro) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`6e8c7b4`](https://github.com/antonioribeiro/google2fa/commit/6e8c7b46d42ebbf0ed116379e6f48234ebd169e4) Add '$encoding' parameter to getQRCodeInline() to avoid CJK issues. ### 📊 Changes **1 file changed** (+6 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/Google2FA.php` (+6 -1) </details> ### 📄 Description If $company or $holder includes CJK(or any UTF8) character, the default encoding ISO8859-1 which used in BaconQrCodeGenerator will cause an error while it generating QR code. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 17:49:20 +03:00
Sign in to join this conversation.
No labels
bug
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/google2fa#344
No description provided.