-
Notifications
You must be signed in to change notification settings - Fork 118
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
fcl forces us to use a less secure content-security-policy #775
Comments
Protobuf issue probably: protocolbuffers/protobuf-javascript#25 |
This also seems to be an issue for using fcl from inside of a browser extension. cc: @gregsantos |
I have the same problem. Any workaround? |
This was merged on Oct 15th and was included in [email protected] Updating google-protobuf version to @^3.19.1 should be able to fix this |
Hmm ok seems like [email protected] still hasn't fixed all CSP unsafe-eval issues |
I think REST api is very soon will solve all this problems. |
@orodio @gregsantos Any update on this? It is preventing the use of the JS SDK in contexts where it would be very useful, such as Cloudflare Workers and browser extensions. Where can I find information/follow progress on the REST api? |
@mikeylemmon The HTTP API has been released on testnet and mainet! https://rest-testnet.onflow.org/v1/ https://rest-mainnet.onflow.org/v1/ The SDK transport modules have moved into their own packages. More details available in the SDK CHANGELOG and on the Docs Site at onflow.org |
Instructions
Please fill out the template below to the best of your ability and include a label indicating which tool/service you were working with when you encountered the problem.
Problem
@onflow/protobuf
callseval
in it. this is why my project is forced to allowunsafe-eval
in our content-security-policy.Steps to Reproduce
npm install
npm run build && npm run start
and expect to see csp errors in the console.fcl
was added...npm run build && npm run start
and the application will execute.Acceptance Criteria
fcl
do not need to allowunsafe-eval
Context
n/a
The text was updated successfully, but these errors were encountered: