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

Issue/2962 Upgrade PapaParse (for ReDos vulnerability) & Fixed HTTP 416 Error #2972

Merged
merged 3 commits into from Sep 18, 2020
Merged

Issue/2962 Upgrade PapaParse (for ReDos vulnerability) & Fixed HTTP 416 Error #2972

merged 3 commits into from Sep 18, 2020

Conversation

t83714
Copy link
Contributor

@t83714 t83714 commented Sep 18, 2020 •
edited

What this PR does

Fixes #2962

  • Upgrade PapaParse to 5.3.0
  • Fixed PapaParse Chunk loading handling issue to avoid HTTP 416 (Range Not Satisfiable) Error
    • It's a regression of version 5.1.0
    • Now use our forked repo
    • Has submitted PR to the official repo

Checklist

  • unit tests aren't applicable
  • I've updated CHANGES.md with what I changed.
  • I've linked this PR to an issue in ZenHub (core dev team only)

@t83714
Copy link
Contributor Author

t83714 commented Sep 18, 2020

tested on CI preview~ Merged~

@t83714 t83714 merged commit 7aeab66 into master Sep 18, 2020
@t83714 t83714 deleted the issue/2962 branch September 18, 2020 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade papaparse to 5.2.0
1 participant
@t83714