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

Critical Security Vulnerability in dependency package "pdfjs" #1109

Closed
bdalvandi-awaremd opened this issue Jun 28, 2024 · 8 comments
Closed

Comments

@bdalvandi-awaremd
Copy link

bdalvandi-awaremd commented Jun 28, 2024

Bug Report or Feature Request (mark with an x)
- [ ] Regression (a behavior that used to work and stopped working in a new release)
- [X] Bug report -> please search issues before submitting
- [ ] Feature request
- [ ] Documentation issue or request

The latest version of ng2-pdf-viewer (10.2.2) has a dependency to pdfjs-dist version 3.11.x which has recently been discovered to have an extremely critical vulnerability, allowing attacks on the domain.
The latest version of pdfjs-dist has remediated that vulnerability, I am wondering if a new version of ng2-pdf-viewer coming out soon that uses the latest version and remediates this vulnerability?

@shamoon
Copy link
Contributor

shamoon commented Jun 28, 2024

Did you search? Multiple issues cover this already

@bdalvandi-awaremd
Copy link
Author

Did you search? Multiple issues cover this already

Yes. But I don't see any of them clearly explaining how to overcome the vulnerability. The closest thing I have seen is ppl mentioning to set the eval to false or something, but where and how is not clear.
Can you by any chance point me to a clear solution? thanks.

@mejobloggs-cw
Copy link

I agree it's not clear how to resolve this security issue. Is there a patch or update coming?

@shamoon
Copy link
Contributor

shamoon commented Jul 3, 2024

Did you try reading the release notes, eg for v10.2.0?

#1092 resolves the CVE but does not update the pdfjs package to 4.x, so automated security tools will still complain even though the issue is resolved.

@DerekLiang
Copy link

Does anyone know what is the effort to update the dependency to pdfjs 4.x?

@shamoon
Copy link
Contributor

shamoon commented Jul 3, 2024

... #1105

@shamoon
Copy link
Contributor

shamoon commented Oct 21, 2024

Should be closed

@VadimDez
Copy link
Owner

Closing @shamoon

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

5 participants