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

update fun-hooks and use no-eval version for CSP #3814

Merged
merged 2 commits into from
May 14, 2019
Merged

Conversation

snapwich
Copy link
Collaborator

@snapwich snapwich commented May 9, 2019

Type of change

  • Refactoring (no functional changes, no api changes)

Description of change

I've updated the fun-hooks library to spit out two versions, one that uses new Function (with better performance) and one that does not but is strict content security policy compliant. This updates Prebid.js to use the CSP compliant one to allow Prebid.js to run on sites with strict CSP.

Other information

Related to #3590

@jsnellbaker jsnellbaker self-requested a review May 13, 2019 12:59
@jsnellbaker jsnellbaker self-assigned this May 13, 2019
@jsnellbaker jsnellbaker added needs 2nd review Core module updates require two approvals from the core team and removed needs review labels May 13, 2019
@jaiminpanchal27 jaiminpanchal27 merged commit a4f247c into master May 14, 2019
jacekburys-quantcast pushed a commit to jacekburys-quantcast/Prebid.js that referenced this pull request May 15, 2019
* update fun-hooks and use no-eval version for CSP

* update fun-hooks to version with proper assign for ie
VideoReach pushed a commit to VideoReach/Prebid.js that referenced this pull request Aug 1, 2019
* update fun-hooks and use no-eval version for CSP

* update fun-hooks to version with proper assign for ie
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs 2nd review Core module updates require two approvals from the core team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants