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

Rule Enforcement: include support for adomain #6986

Merged
merged 20 commits into from
Jun 10, 2021
Merged

Conversation

ChrisHuie
Copy link
Collaborator

No description provided.

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request fixes 1 alert when merging c702558 into 0b55d6b - view on LGTM.com

fixed alerts:

  • 1 for Improper code sanitization

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request fixes 1 alert when merging 456033b into 0b55d6b - view on LGTM.com

fixed alerts:

  • 1 for Improper code sanitization

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request fixes 1 alert when merging b811741 into 0b55d6b - view on LGTM.com

fixed alerts:

  • 1 for Improper code sanitization

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request fixes 54 alerts when merging 1fffadf into 0b55d6b - view on LGTM.com

fixed alerts:

  • 50 for Useless regular-expression character escape
  • 1 for Useless conditional
  • 1 for Client-side URL redirect
  • 1 for Improper code sanitization
  • 1 for Unneeded defensive code

Copy link
Collaborator

@patmmccann patmmccann left a comment

Choose a reason for hiding this comment

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

please restore widespace

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request fixes 54 alerts when merging ab5a648 into 0b55d6b - view on LGTM.com

fixed alerts:

  • 50 for Useless regular-expression character escape
  • 1 for Useless conditional
  • 1 for Client-side URL redirect
  • 1 for Improper code sanitization
  • 1 for Unneeded defensive code

@@ -61,7 +61,7 @@ export const spec = {
'gdprCmp': bidderRequest && bidderRequest.gdprConsent ? 1 : 0,
'hb': '1',
'hb.cd': CUST_DATA ? encodedParamValue(CUST_DATA) : '',
'hb.floor': '',
'hb.floor': bid.bidfloor || '',
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think you got the old widespace.js

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request fixes 54 alerts when merging ccfa44b into 0b55d6b - view on LGTM.com

fixed alerts:

  • 50 for Useless regular-expression character escape
  • 1 for Useless conditional
  • 1 for Client-side URL redirect
  • 1 for Improper code sanitization
  • 1 for Unneeded defensive code

@gglas gglas self-requested a review June 10, 2021 01:39
Copy link

@gglas gglas left a comment

Choose a reason for hiding this comment

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

lgtm

@patmmccann patmmccann merged commit 31d1d0e into prebid-5.0 Jun 10, 2021
@patmmccann patmmccann deleted the ChrisHuie-adomain branch June 10, 2021 01:48
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.

3 participants