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

GDPR-compliance for Prebid Server cookiesyncs #2516

Closed
dbemiller opened this issue May 10, 2018 · 6 comments
Closed

GDPR-compliance for Prebid Server cookiesyncs #2516

dbemiller opened this issue May 10, 2018 · 6 comments
Assignees

Comments

@dbemiller
Copy link
Contributor

dbemiller commented May 10, 2018

Prebid.js cookie syncs with Prebid Server violate GDPR. See prebid/prebid-server#501 for more details.

If the GDPR module is included, please include the following properties in the POST /cookie_sync request body to Prebid Server:

{
  "gdpr": 1,
  "gdpr_consent": "vendor-consent-string"
}

If GDPR is in effect:

If GDPR is not in effect:

  • gdpr should be 0
  • gdpr_consent should be undefined

If the GDPR status is uncertain:

  • gdpr should be undefined
  • gdpr_consent is optional, but you should send it if you have it.

If the GDPR module is not included, then the status is uncertain. gdpr should be undefined, and gdpr_consent is optional.

Let me know if you have any questions or concerns.

Thanks!

@bretg
Copy link
Collaborator

bretg commented May 10, 2018

While we're address this, can we cover #2242 ? I think this is the same piece of code. We haven't been able to get to it.

@jsnellbaker
Copy link
Collaborator

@dbemiller if the module is disabled, does that fall into the 'GDPR is not in effect' bucket or should we just not add the gdpr and gdpr_consent fields at all to the POST request?

@jsnellbaker
Copy link
Collaborator

@bretg I will aim to include that issue you noted as part of my changes here.

@dbemiller
Copy link
Contributor Author

dbemiller commented May 11, 2018

@jsnellbaker in that case, don't add the gdpr or gdpr_consent fields at all to the POST request.

I updated the top post to reflect this. Thanks for the catch!

@dbemiller
Copy link
Contributor Author

@jsnellbaker safe to close this?

@jsnellbaker
Copy link
Collaborator

Yes - this change was merged in as part of the 1.11.0 release.

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

No branches or pull requests

3 participants