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

Does Not Support All Unicode Chars #2756

Closed
elliot-huffman opened this issue Aug 28, 2023 · 1 comment · Fixed by #2758
Closed

Does Not Support All Unicode Chars #2756

elliot-huffman opened this issue Aug 28, 2023 · 1 comment · Fixed by #2758
Assignees
Labels
priority:p1 High priority but not blocking. Causes major but not critical loss of functionality SLA <=7days. type:bug Something isn't working

Comments

@elliot-huffman
Copy link

elliot-huffman commented Aug 28, 2023

Describe the bug
When putting a URL into the "Query sample input" text field (the address bar of the app), if that URL has Unicode characters in it, the whole app crashes and removes itself from the DOM and goes to a pseudo white screen.

To Reproduce
Steps to reproduce the behavior:

  1. Position your cursor in the address bar of the app
  2. Put ANY emoji in it, in any position

Expected behavior
The emoji stays in the URL bar and can be queried against since Entra ID supports full Unicode chars in their properties.
The app doesn't crash ultimately.

Screenshots
(sensitive data is redacted by photo edits)
Before Emoji in the address bar is injected:
Screenshot of the query bar showing a filtered group with an emoji prompt pulled up but no emoji.

After:
Screenshot of the App's crashed state

Desktop (please complete the following information):

  • OS: Windows 11, 22H2
  • Browser Microsoft Edge
  • Version 116.0.1938.62

Additional context
console logs:

react-dom.production.min.js:4014  Error: parser.js: initializeInputChars(): input length out of range: 180
    at N (apg-api-bundle.js:5785:115)
    at t.exports.parse (apg-api-bundle.js:5916:21)
    at e.value (abnf.ts:43:40)
    at auto-complete.util.ts:62:32
    at Pf (auto-complete.util.ts:50:17)
    at Zf (AutoComplete.tsx:290:24)
    at ka (react-dom.production.min.js:3436:7)
    at Ts (react-dom.production.min.js:4165:7)
    at ku (react-dom.production.min.js:6626:86)
    at bl (react-dom.production.min.js:6283:11)
hs @ react-dom.production.min.js:4014
n.callback @ react-dom.production.min.js:4031
Uo @ react-dom.production.min.js:2972
_u @ react-dom.production.min.js:5670
bu @ react-dom.production.min.js:5635
Au @ react-dom.production.min.js:5610
(anonymous) @ react-dom.production.min.js:6365
Cl @ react-dom.production.min.js:6324
ll @ react-dom.production.min.js:6039
zi @ react-dom.production.min.js:2538
dl @ react-dom.production.min.js:6061
Le @ react-dom.production.min.js:736
Wr @ react-dom.production.min.js:2087
qt @ react-dom.production.min.js:1285
jt @ react-dom.production.min.js:1267
apg-api-bundle.js:5785  Uncaught Error: parser.js: initializeInputChars(): input length out of range: 180
    at N (apg-api-bundle.js:5785:115)
    at t.exports.parse (apg-api-bundle.js:5916:21)
    at e.value (abnf.ts:43:40)
    at auto-complete.util.ts:62:32
    at Pf (auto-complete.util.ts:50:17)
    at Zf (AutoComplete.tsx:290:24)
    at ka (react-dom.production.min.js:3436:7)
    at Ts (react-dom.production.min.js:4165:7)
    at ku (react-dom.production.min.js:6626:86)
    at bl (react-dom.production.min.js:6283:11)
N @ apg-api-bundle.js:5785
parse @ apg-api-bundle.js:5916
value @ abnf.ts:43
(anonymous) @ auto-complete.util.ts:62
Pf @ auto-complete.util.ts:50
Zf @ AutoComplete.tsx:290
ka @ react-dom.production.min.js:3436
Ts @ react-dom.production.min.js:4165
ku @ react-dom.production.min.js:6626
bl @ react-dom.production.min.js:6283
yl @ react-dom.production.min.js:6274
vl @ react-dom.production.min.js:6258
al @ react-dom.production.min.js:5978
ll @ react-dom.production.min.js:6033
zi @ react-dom.production.min.js:2538
dl @ react-dom.production.min.js:6061
Le @ react-dom.production.min.js:736
Wr @ react-dom.production.min.js:2087
qt @ react-dom.production.min.js:1285
jt @ react-dom.production.min.js:1267
defaultWorkerFactory.js:23  Refused to create a worker from 'https://developer.microsoft.com/json.worker.js' because it violates the following Content Security Policy directive: "worker-src data:".

(anonymous) @ defaultWorkerFactory.js:23
e @ defaultWorkerFactory.js:66
value @ defaultWorkerFactory.js:124
n @ simpleWorker.js:238
value @ editorWorkerServiceImpl.js:545
value @ editorWorkerServiceImpl.js:557
value @ webWorker.js:50
value @ webWorker.js:85
e._getClient @ workerManager.js:55
e.getLanguageServiceWorker @ workerManager.js:66
i @ jsonMode.js:19
e._resetSchema @ languageFeatures.js:73
(anonymous) @ languageFeatures.js:41
value @ event.js:566
value @ modelServiceImpl.js:445
(anonymous) @ modelServiceImpl.js:308
(anonymous) @ modelServiceImpl.js:71
value @ event.js:566
value @ textModel.js:266
(anonymous) @ index.mjs:373
tu @ react-dom.production.min.js:5051
ru @ react-dom.production.min.js:5176
wl @ react-dom.production.min.js:6418
(anonymous) @ react-dom.production.min.js:6347
C @ scheduler.production.min.js:94
I @ scheduler.production.min.js:125
simpleWorker.js:27  Failed to construct 'Worker': Access to the script at 'https://developer.microsoft.com/json.worker.js' is denied by the document's Content Security Policy.
He @ simpleWorker.js:27
value @ editorWorkerServiceImpl.js:547
value @ editorWorkerServiceImpl.js:557
value @ webWorker.js:50
value @ webWorker.js:85
e._getClient @ workerManager.js:55
e.getLanguageServiceWorker @ workerManager.js:66
i @ jsonMode.js:19
e._resetSchema @ languageFeatures.js:73
(anonymous) @ languageFeatures.js:41
value @ event.js:566
value @ modelServiceImpl.js:445
(anonymous) @ modelServiceImpl.js:308
(anonymous) @ modelServiceImpl.js:71
value @ event.js:566
value @ textModel.js:266
(anonymous) @ index.mjs:373
tu @ react-dom.production.min.js:5051
ru @ react-dom.production.min.js:5176
wl @ react-dom.production.min.js:6418
(anonymous) @ react-dom.production.min.js:6347
C @ scheduler.production.min.js:94
I @ scheduler.production.min.js:125
editorSimpleWorker.js:810  Uncaught (in promise) Error: Unexpected usage
    at e.value (editorSimpleWorker.js:810:29)
    at webWorker.js:52:24
@HotCakeX
Copy link

I can reproduce it too
Windows 11 22H2
Edge browser 116

Step 1: navigate to this page: https://developer.microsoft.com/en-us/graph/graph-explorer

Step 2: put an emoji in here

image

@thewahome thewahome self-assigned this Aug 29, 2023
@thewahome thewahome added type:bug Something isn't working priority:p1 High priority but not blocking. Causes major but not critical loss of functionality SLA <=7days. labels Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:p1 High priority but not blocking. Causes major but not critical loss of functionality SLA <=7days. type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants