[PR #29] [MERGED] fix: remove unicode control characters when attempting to parse JSON #27

New issue

Closed

opened 2026-03-04 12:07:06 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 12:07:06 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/4IceG/luci-app-modemdata/pull/29
Author: @WereCatf
Created: 12/17/2025
Status: ✅ Merged
Merged: 12/17/2025
Merged by: @4IceG

Base: main ← Head: unicodecontrolchars

📝 Commits (1)

e3e1545 fix: remove unicode control characters when attempting to parse JSON

📊 Changes

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

View changed files

📝 luci-app-modemdata/htdocs/luci-static/resources/view/modem/modempreview.js (+2 -2)

📄 Description

Some operators appear to be inserting unicode control characters at the end of their name which then causes attempts to parse the raw text into proper JSON in modempreview.js to fail. As an example the author of this bugfix sees operator name reported as "evz\u001e\u0006".

This bugfix removes any unicode control characters from the raw text before attempting to parse it.

_{🔄 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/4IceG/luci-app-modemdata/pull/29 **Author:** [@WereCatf](https://github.com/WereCatf) **Created:** 12/17/2025 **Status:** ✅ Merged **Merged:** 12/17/2025 **Merged by:** [@4IceG](https://github.com/4IceG) **Base:** `main` ← **Head:** `unicodecontrolchars` --- ### 📝 Commits (1) - [`e3e1545`](https://github.com/4IceG/luci-app-modemdata/commit/e3e1545cad4519396bf954c9bc2a61a142d4b6e5) fix: remove unicode control characters when attempting to parse JSON ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `luci-app-modemdata/htdocs/luci-static/resources/view/modem/modempreview.js` (+2 -2) </details> ### 📄 Description Some operators appear to be inserting unicode control characters at the end of their name which then causes attempts to parse the raw text into proper JSON in modempreview.js to fail. As an example the author of this bugfix sees operator name reported as "evz\u001e\u0006". This bugfix removes any unicode control characters from the raw text before attempting to parse it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>