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

Vulnerability / Security issue on playcanvas stable min js file #6828

Open
nehatdedinca opened this issue Jul 17, 2024 · 1 comment
Open

Comments

@nehatdedinca
Copy link

Description

I am using PlayCanvas through downloading .zip folder and then integrating it in my application by pointing to correct animation/illustrations assets.

After regression (PEN) testing in my application, there is the result report which indicates that the file playcanvas-stable.min.js contains Unicode bidirectional (BiDi) control characters which could lead to some security issues (vulnerabilities/breaches). In more details I have attached the screenshot from the report with more elaboration towards the issue.

Screenshot 2024-07-17 at 3 44 40 PM

Moreover, I have tried reproducing the issue, through BiDi character checkers such as using NPM packages bidi-js, it found the vulnerabilities (fishy characters) but not able to solve as per the minified file. I was not able to get the non-minified file.

This is a list of BiDi characters. Most, if not all, of the character breaches found were under the category of Other Neutral.

Please try to reproduce the issue by running regression or pen tests on an application which uses playcanvas (playcanvas-stable.min.js) and the vulnerability should be there.

Any suggestion on how to correct overcome this issue would be highly appreciated. Thank you in advance! ND

@willeastcott
Copy link
Contributor

It appears that Terser is somehow introducing BiDi characters into playcanvas.min.js. They are not present in playcanvas.js. I have high confidence that the BiDi characters are harmless, but if you are concerned, feel free to swap out playcanvas.min.js with playcanvas.js.

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

No branches or pull requests

2 participants