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

Add pubProvided to userId.md #2357

Merged
merged 9 commits into from
Sep 30, 2020
Merged

Add pubProvided to userId.md #2357

merged 9 commits into from
Sep 30, 2020

Conversation

jdwieland8282
Copy link
Member

adding documentation for the Pubprovided user id module

adding documentation for the Pubprovided user id module
jdwieland8282 and others added 3 commits September 22, 2020 13:18
changed DMP to lowercase dmp in stype supported values comment
@bretg bretg changed the title Update userId.md Add pubProvided to userId.md Sep 22, 2020

- dmp - this uid comes from the in-page dmp named in eids.source
- ppuid - this uid comes from the publisher named in eids.source
- sha256email - this uid is based on an email hash from the entity named in eids.source
Copy link
Collaborator

Choose a reason for hiding this comment

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

@jdwieland8282 maybe this is a discussion for identity committee, but do we really want to support passing the actual hashed email through the bidstream like this? a hashed email is like a fingerprint in that you can't "reset" it - once you have it, you will always have the same link to a set of user data, even if they've asked someone upstream to reset/clear their data. since there's no efficient way (today) to tell EVERY platform in the industry to wipe data for a user, the best way today is simply to generate a new user id. this is just like apple and android allowing you to reset your MAID. identity providers that base an ID off of a hashed email are fine since they can change the ID they generate if the user has asked to reset/opt out at some point.

my suggestion for now is to not include mentions of passing sha256email directly today, and we can discuss in committee meeting next week to see if anyone else agrees with me or if i'm just being overly precautious. thoughts?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thats a fair @smenzer, I can strip that example out for now. I would like to have this discussion in 5775 though. I think defining what types of ids a given user id module generates is worth while.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thats a fair @smenzer, I can strip that example out for now. I would like to have this discussion in 5775 though. I think defining what types of ids a given user id module generates is worth while.

ok cool, i'll copy my comment above to that issue for discussion...thanks, @jdwieland8282 !

Copy link
Collaborator

@smenzer smenzer left a comment

Choose a reason for hiding this comment

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

lgtm

adding language about the userid.pubProvided value and userIdAsEids value/function
@bretg bretg merged commit 711b500 into master Sep 30, 2020
@bretg bretg deleted the jdwieland8282-patch-2 branch September 30, 2020 15:43
bretg added a commit that referenced this pull request Sep 30, 2020
* reverted optanon in header (#2315)

Co-authored-by: Steve <ssuranie@appnexus.com>

* LiveIntent registration link fix (#2300)

* Add LiveIntent ID module to the overview

* Update LiveIntent ID docs

- remove 'providedIdentifierName' parameter, as it's removed from code (prebid/Prebid.js#5407)
- adjust links in the docs
- new privacy policy
- code examples updated to include the storage part

* typo fix

* added registration information

* LiveIntent registration link fix

* Update oftmedia.md (#2308)

forgot to insert the code " schain_supported: true", same as appnexus. thanks!

* Delete pb-is-amp.html (#2311)

Duplicate file with trailing space after the name

* Update invibes.md (#2312)

* parrableIdSystem: Add an optional timezone and timezone offset allow/block filter (#2184)

* PBID-11: Update Parrable ID system configuration example to remove storage options

* PBID-25: Add docs for Parrable ID timezone filtering feature config

* PBID-25: Fix grammar in userId timezone filtering

* add native support for ablida bid adapter (#2164)

* Blue Billywig bid adapter - add video param to docs (#2198)

* add Blue Billywig bidder adaptor dev docs

* Add missing empty line

* move empty line to right place...

Co-authored-by: Klaas-Jan Boon <klaas-janboon@live.nl>
Co-authored-by: Klaas-Jan Boon <klaas-janboon@ip-172-16-224-85.eu-west-1.compute.internal>

* Document auctionDebug (#2255)

* Document auctionDebug

* Update integrate-with-the-prebid-analytics-api.md

* Fix typo (#2319)

* added adapter request params (#2318)

* Livewrapped support for native and video (#2323)

* Livewrapped bidder adapter

* Added analytics documentation

* Spelling

* Added support for tcf2 and schain

* video and native support

* update based on new prebid.org site content (#2321)

* update based on new prebid.org site content

* added link to prebid.org and updated blog url

* fixing old PBS+video message (#2327)

* adding CMP link to other places (#2328)

* moving cookie sync PBS FAQ (#2330)

* Added docs.prebid.org google analytics tag

* added debug logging endpoints (#2336)

* first party data docs (#2325)

* first party data docs

* adding PBS-Java note

* minor edits

Co-authored-by: Jean Stemp <jstemp@appnexus.com>

* Update indexExchange.md to include Prebid Server details (#1) (#2331)

* Update indexExchange.md

* minor spelling fix

* Fixed minor grammatical errors (#2)

* Changed timeout example to 50ms

Co-authored-by: rimaburder-index <55195208+rimaburder-index@users.noreply.github.com>

Co-authored-by: rimaburder-index <55195208+rimaburder-index@users.noreply.github.com>

* Revert "Livewrapped support for native and video (#2323)" (#2324)

This reverts commit 1bcb343.

* Add Quantcast userId submodule documentation (#2339)

* adding InMobi docs (#2326)

* added nurl faq (#2345)

* added changes for Dr.Prebid (#2344)

* brought releases up to 4.8 (#2349)

* added mopub set up for rewarded videos (#2342)

* added mopub set up for rewarded videos

* Added full screen video mopub set up

* Fixed markdown error

* Made changes to XML code per dev

* Made edits per jstemp

* Fix alt descriptions

* fixed alt text

Co-authored-by: Steve <ssuranie@appnexus.com>

* Add gdpr supported in ablida docs (#2334)

* add native support for ablida bid adapter

* add gdpr supported

* doc: add getEvents method to the public API  (#2322)

* feat: add getEvents method to the public API prebid/Prebid.js#5703

* Update publisher-api-reference.md

* Audigent HaloId Documentation (#2320)

* Update show-outstream-video-ads.md (#2303)

* New adapter "Cointraffic" added (#2301)

* Add additional parameters to config (#2298)

Co-authored-by: Yevhenii Tykhostup <yevhenii.tykhostup@audienceproject.com>

* MediaNet SChain Support (#2294)

* add Pub-X dev-docs (#2287)

* added docs for instreamTracking (#2277)

Co-authored-by: monis.q <monis.q@media.net>

* Prebid SDK 1.8: bid response callback and OMSDK (#2340)

* Prebid SDK 1.8: bid response callback and OMSDK

* prebidSDK1_8: updated language

* Added TCF v2.0 support to the documentation (#2350)

* Indicate new features in AMX RTB 1.1 Update (#2221)

* indicate tcf2, user ID, coppa support for AMX RTB bid adapter

* document ad unit ID

* renaming waardex to waardex.ak (#2352)

* Update userId.md (#2193)

Provides documentation for prebid/Prebid.js#5577

Co-authored-by: bretg <bgorsline@gmail.com>

* Markdown docs for both gjirafa and malltv bid adapters (#2200)

* added markdowns for gjirafa and malltv

* quotes

* added 'video' to supported media types

* Add Supply Chain Support to Docs (#2354)

* Prebid sdk1.8 (#2356)

* Prebid SDK 1.8: bid response callback and OMSDK

* prebidSDK1_8: updated language

* prebidSDK1_8: typos

* misc user ID file fixes (#2358)

* Update download.md

* Update download.md

removing pubprovided from download page

* adding note that size_id is required for PBS (#2363)

* adds new params to gumgum doc (#2332)

* welect: update docu to tcf2.0 (#2233)

* update docu to tcf2.0

* add gvlid in metadata

* No bid version 1.2.7 (#2253)

* Adde support for CCPA

* Support for TCF 2.0

Co-authored-by: Reda Guermas <reda.guermas@nobid.io>

* ConnectAd docs update (#2305)

* Update invibes.md (#2361)

* wordsmithing PBS overview (#2367)

* Updating Brightroll adapter details (#2365)

* Floors update (#2371)

* Prebid SDK 1.8: bid response callback and OMSDK

* prebidSDK1_8: updated language

* prebidSDK1_8: typos

* Floors: added floor providers

* fix quantcast download (#2373)

* removing PBS adapters from download page (#2374)

* Adding pubx.ai to the list of dynamic floor providers (#2375)

Co-authored-by: Alex Rosen <alex@platform360.co>

* Added md for new adaptor jixie (#2338)

* Separated descriptions about Adform Prebid.js and Adform Prebid Server adapters (#2372)

Co-authored-by: Justas Pupelis <justas.pupelis@adform.com>

* GumGum: updates doc to include iriscat field (#2376)

* specifies us privacy is supported

* adds video support to gumgum md

* adds support for tcf 2

* adds new bid param to gumgum.md

* updates gumgum doc

* fix adtarget link (#2377)

* fix adtarget link (#2378)

* adding guideline for CMP loading (#2379)

* Add adnow bidder adaptor doc (#2314)

* Add adnow bidder adaptor doc

* Remove unnecessary param mediaType

* Add adrelevantis bid adapter doc (#2329)

* Create adrelevantis.md

* Update adrelevantis.md

* Native support for Logicad adapter dev-docs (#2335)

* Adagio: add support for CCPA/COPPA (#2337)

* Smaato: Docs for optional in-app webview params (#2347)

* Add params to intentIq (#2351)

* Update Sublime adapter doc for v0.6 (#2353)

* Update bridgewell.md (#2359)

Add the description for the new bid parameter.

* Add video support in ablida docs (#2362)

* add native support for ablida bid adapter

* add gdpr supported

* add video ad support

* SpotX adapter: add page param (#2364)

* Add pubProvided to userId.md (#2357)

* Update userId.md

adding documentation for the Pubprovided user id module

* Update userId.md

changed DMP to lowercase dmp in stype supported values comment

* typos, wordsmithing

* added note about stype

* Update download.md

adding support to download.md

* Update userId.md

* Update userId.md

adding language about the userid.pubProvided value and userIdAsEids value/function

Co-authored-by: bretg <bgorsline@gmail.com>

* Updated Brightroll adapter documentation (#2381)

* Updating Brightroll adapter details

* updated doc

Co-authored-by: MartianTribe <steve@martiantribe.com>
Co-authored-by: Steve <ssuranie@appnexus.com>
Co-authored-by: amoltchan <56824198+amoltchan@users.noreply.github.com>
Co-authored-by: murilodiko <murilomcozza@gmail.com>
Co-authored-by: AaronColbyPrice <67345931+AaronColbyPrice@users.noreply.github.com>
Co-authored-by: invibes <51820283+invibes@users.noreply.github.com>
Co-authored-by: Ian Flournoy <ian@flournoy.us>
Co-authored-by: Dan <webmaster@kakao-karten.de>
Co-authored-by: Klaas-Jan Boon <awisoewas@gmail.com>
Co-authored-by: Klaas-Jan Boon <klaas-janboon@live.nl>
Co-authored-by: Klaas-Jan Boon <klaas-janboon@ip-172-16-224-85.eu-west-1.compute.internal>
Co-authored-by: Patrick McCann <patmmccann@gmail.com>
Co-authored-by: Seba Perez <sperez@est.frba.utn.edu.ar>
Co-authored-by: bjorn-lw <32431346+bjorn-lw@users.noreply.github.com>
Co-authored-by: Jean Stemp <38964447+jeanstemp@users.noreply.github.com>
Co-authored-by: Jean Stemp <jstemp@appnexus.com>
Co-authored-by: Rob Hazan <robhazan@gmail.com>
Co-authored-by: rimaburder-index <55195208+rimaburder-index@users.noreply.github.com>
Co-authored-by: Mehmet Can Kurt <mcankurt@quantcast.com>
Co-authored-by: Daniel Lawrence <daniel.lawrence@inmobi.com>
Co-authored-by: ppuviarasu <ppuviarasu@appnexus.com>
Co-authored-by: gpolaert <gpolaert@pubstack.io>
Co-authored-by: Anthony Lauzon <anthony.lauzon@gmail.com>
Co-authored-by: Aleksandr Štšepelin <as@jp.ee>
Co-authored-by: frstua <moyade@gmail.com>
Co-authored-by: Yevhenii Tykhostup <yevhenii.tykhostup@audienceproject.com>
Co-authored-by: Rahul Shandilya <67756716+c3p-0@users.noreply.github.com>
Co-authored-by: Pub-X <63354024+Pub-X@users.noreply.github.com>
Co-authored-by: Monis Qadri <monis0395@users.noreply.github.com>
Co-authored-by: monis.q <monis.q@media.net>
Co-authored-by: Bryan Szekely <49168870+bszekely1@users.noreply.github.com>
Co-authored-by: thuyhq <61451682+thuyhq@users.noreply.github.com>
Co-authored-by: Nick Jacob <nickbjacob@gmail.com>
Co-authored-by: angelamerkelprebid <69318193+angelamerkelprebid@users.noreply.github.com>
Co-authored-by: Drilon Kastrati <drilon@gjirafa.com>
Co-authored-by: Mirko Feddern <3244291+mirkorean@users.noreply.github.com>
Co-authored-by: jdwieland8282 <wieland.jeff@gmail.com>
Co-authored-by: susyt <susan@fox.camp>
Co-authored-by: Nick Duitz <42961155+nduitz@users.noreply.github.com>
Co-authored-by: redaguermas <redaguermas@gmail.com>
Co-authored-by: Reda Guermas <reda.guermas@nobid.io>
Co-authored-by: rtuschkany <35923908+rtuschkany@users.noreply.github.com>
Co-authored-by: smithaammassamveettil <39389834+smithaammassamveettil@users.noreply.github.com>
Co-authored-by: alexpubx <71898905+alexpubx@users.noreply.github.com>
Co-authored-by: Alex Rosen <alex@platform360.co>
Co-authored-by: jxdeveloper1 <71084096+jxdeveloper1@users.noreply.github.com>
Co-authored-by: Justas Pupelis <pupelis.justas@gmail.com>
Co-authored-by: Justas Pupelis <justas.pupelis@adform.com>
Co-authored-by: vingood <antispamer@gmail.com>
Co-authored-by: ghguo <gary_guo@hotmail.com>
Co-authored-by: logicad <ytsuchiya@so-netmedia.jp>
Co-authored-by: Olivier <osazos@adagio.io>
Co-authored-by: Stephan Brosinski <s@brosinski.com>
Co-authored-by: yuvalgg <57989766+yuvalgg@users.noreply.github.com>
Co-authored-by: Jeremie Patonnier <jeremie.patonnier@gmail.com>
Co-authored-by: Rigel Cheng <rigel.cheng@gmail.com>
Co-authored-by: Amanda Dillon <41923726+agdillon@users.noreply.github.com>
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