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

CSP Violation for dependency Restructure/src/Pointer.js:line 229 #1451

Closed
djaffer opened this issue Aug 10, 2021 · 11 comments · Fixed by #1908
Closed

CSP Violation for dependency Restructure/src/Pointer.js:line 229 #1451

djaffer opened this issue Aug 10, 2021 · 11 comments · Fixed by #1908
Labels

Comments

@djaffer
Copy link

djaffer commented Aug 10, 2021

I am not able to use strict CSP without unsafe-eval. Can you please fix the dependency being used?

image

image

@jeetiss
Copy link
Collaborator

jeetiss commented Aug 10, 2021

@diegomura maybe it's time to update Restructure? I can setup babel for transpiling this package to es5, as I understand only this blocks for update!

@diegomura
Copy link
Owner

That sounds good! I'm not familiarized with what's needed. Seems you have a clearer idea so I'll trust you 😄

@diegomura diegomura added the bug label Aug 10, 2021
@djaffer
Copy link
Author

djaffer commented Sep 9, 2021

any update on this please @diegomura @jeetiss

@jeetiss
Copy link
Collaborator

jeetiss commented Dec 18, 2021

@djaffer my plan doesn't help with CSP because @react-pdf has another eval in @react-pdf/yoga and it's not so easy to fix eval in it. you can read more at #510

P.S I'm still planning to upgrade Restructure to the latest version because it doesn't use an obsolete buffer constructor. I will do it when PR #1609 is merged

@twistedpair
Copy link

Looks like #1609 has been successfully merged. Any movement on addressing these CSP blocking issues? Thanks.

@seanquinn
Copy link

+1 on this

@jacek-seliga-fat-ninja
Copy link

+1 on this @diegomura

@maelgff
Copy link

maelgff commented Apr 14, 2022

+1 on this 🙏

1 similar comment
@Serbiaguy96
Copy link

Serbiaguy96 commented Apr 27, 2022

+1 on this 🙏

carlobeltrame added a commit to carlobeltrame/react-pdf that referenced this issue Jun 16, 2022
carlobeltrame added a commit to carlobeltrame/react-pdf that referenced this issue Jun 16, 2022
carlobeltrame added a commit to carlobeltrame/react-pdf that referenced this issue Jun 16, 2022
diegomura pushed a commit to carlobeltrame/react-pdf that referenced this issue Jul 2, 2022
diegomura pushed a commit to carlobeltrame/react-pdf that referenced this issue Jul 4, 2022
diegomura pushed a commit to carlobeltrame/react-pdf that referenced this issue Jul 11, 2022
diegomura pushed a commit to carlobeltrame/react-pdf that referenced this issue Jul 11, 2022
@djaffer
Copy link
Author

djaffer commented Jul 17, 2022

Is this fixed merged to recent release?

@jepek
Copy link

jepek commented Aug 8, 2022

Hi @diegomura,

Same question from me - has version bump up for reconstruct package in the fontkit already been released?

Without it, I have a problem with the CSP strict rules and I cannot use your great package in production.

Thank you for your work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants