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.

Sign up for GitHub

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

Jump to bottom

Upgrade SheetJS to solve vulnerability to Regular Expression Denial of Service (ReDoS). #9183

Open
gigincg opened this issue Nov 22, 2024 · 3 comments · May be fixed by #9323
Open

Upgrade SheetJS to solve vulnerability to Regular Expression Denial of Service (ReDoS). #9183

gigincg opened this issue Nov 22, 2024 · 3 comments · May be fixed by #9323
Assignees
@noufalrahim
Labels
work-in-progress

Comments

@gigincg
Copy link
Member

gigincg commented Nov 22, 2024

Follow https://docs.sheetjs.com/docs/getting-started/installation/nodejs to switch from npm to sheetjs cdn, for the current implementation, potentially plan for moving out of sheetjs
@Tanuj1718
Copy link
Contributor

@gigincg can you please explain what we are planning to do?

@github-actions github-actions bot added needs-triage question Further information is requested labels Nov 26, 2024
@nihal467 nihal467 removed question Further information is requested needs-triage labels Nov 29, 2024
@gigincg
Copy link
Member Author

gigincg commented Nov 29, 2024 •
edited
Loading

@Tanuj1718 please read the specified link. We are hoping to solve the outdated dependency being used in Care

@gigincg gigincg removed their assignment Nov 29, 2024
@rithviknishad rithviknishad mentioned this issue Dec 3, 2024
Open
6 tasks
@noufalrahim
Copy link
Contributor

@rithviknishad Can u assign this to me

@github-project-automation github-project-automation bot moved this to Triage in Care Dec 5, 2024
@rithviknishad rithviknishad moved this from Triage to Up Next in Care Dec 5, 2024
@noufalrahim noufalrahim linked a pull request Dec 6, 2024 that will close this issue
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@noufalrahim noufalrahim

Labels
work-in-progress
Projects
Care
Status: Up Next
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: changed from npm to SheetJS cdn noufalrahim/care_fe
4 participants
@gigincg @nihal467 @noufalrahim @Tanuj1718