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

Bump seedrandom to avoid unsafe-eval issues #6440

Merged
merged 11 commits into from
Jun 28, 2022
Merged

Bump seedrandom to avoid unsafe-eval issues #6440

merged 11 commits into from
Jun 28, 2022

Conversation

s-mustafa
Copy link
Contributor

@s-mustafa s-mustafa commented May 24, 2022

Hi,

seedrandom@2.4.3 breaks when Content Security Policy is enabled:

Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self'".

It was fixed in davidbau/seedrandom#65

To see the logs from the Cloud Build CI, please join either our discussion or announcement mailing list.


This change is Reviewable

@rthadur rthadur requested review from pyu10055 and lina128 May 24, 2022 16:26
Copy link
Collaborator

@lina128 lina128 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @pyu10055)

Copy link
Collaborator

@pyu10055 pyu10055 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

Reviewed 9 of 9 files at r1, all commit messages.
Reviewable status: :shipit: complete! 2 of 1 approvals obtained

@lina128
Copy link
Collaborator

lina128 commented Jun 9, 2022

Hi @s-mustafa , thank you for your contribution! Can you resolve conflicts? Then the PR is ready to merge.

@s-mustafa
Copy link
Contributor Author

Hi @s-mustafa , thank you for your contribution! Can you resolve conflicts? Then the PR is ready to merge.

Hi @lina128, thanks for the review. I resolved the conflicts

@s-mustafa
Copy link
Contributor Author

Hi @lina128, @pyu10055, possible to merge before there are other conflicts please?

@pyu10055 pyu10055 merged commit b83e5b4 into tensorflow:master Jun 28, 2022
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

Successfully merging this pull request may close these issues.

4 participants