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

100% cpu cost and lagging #17

Open
huunghia98 opened this issue Jun 22, 2022 · 3 comments
Open

100% cpu cost and lagging #17

huunghia98 opened this issue Jun 22, 2022 · 3 comments
Assignees
Labels
question Further information is requested

Comments

@huunghia98
Copy link

In production I have issue with this html.
If use dangerouslySetInnerHTML, there is no error, but if use react-letter CPU will hold on 100% and the browser will be unresponded
Please check
error.zip
.

@mat-sz
Copy link
Owner

mat-sz commented Jun 22, 2022

Thank you for your bug report, this actually allowed me to discover a security issue within lettersanitizer.

Doing yarn upgrade/npm upgrade or removing and reinstalling react-letter should resolve the issue now. Please let me know if the problem persists.

@mat-sz mat-sz closed this as completed Jun 22, 2022
@e10jon
Copy link

e10jon commented Jun 10, 2024

I am getting this same issue now, and I'm not sure how to debug it because the browser crashes. How should I go about debugging this?

@mat-sz
Copy link
Owner

mat-sz commented Jun 10, 2024

@e10jon Could you tell me the exact props you're providing to the Letter component? I'll check.

@mat-sz mat-sz reopened this Jun 10, 2024
@mat-sz mat-sz added the question Further information is requested label Jun 10, 2024
@mat-sz mat-sz self-assigned this Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants