-
-
Notifications
You must be signed in to change notification settings - Fork 908
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
[breaking] Update pdfjs-dist to 4.2.67 #1774
Conversation
This PR is currently breaking most setups that do not support top level await (including Vite - see code changes in test app in this PR). I consider this a deal breaker at the moment and therefore postpone merging it. See mozilla/pdf.js#17349 for more details. |
824f97e
to
611a986
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
@wojtekmaj Unfortunately, after updating react-pdf to version 8.0.2, the dependency pdfjs-dist is version 3.11.174 which is a vulnerable version and fails npm audits. Can pdfjs-dist be upgraded to 4.2.67? Thank you. pdfjs-dist vulnerability |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
56a02ce
to
c15279d
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
d5c2c3b
to
f4d4d49
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
Please read my announcement regarding security vulnerability in #1786 and please continue discussion regarding it there if needed. |
fcbcdca
to
a6d74fb
Compare
a6d74fb
to
a103077
Compare
Superseded by #1809 |
Closes #1664
Breaking changes (on top of #1690):
Blockers: