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 docs for SharedID #1923

Closed
wants to merge 1 commit into from
Closed

add docs for SharedID #1923

wants to merge 1 commit into from

Conversation

jdwieland8282
Copy link
Member

No description provided.

@jdwieland8282 jdwieland8282 requested a review from bretg April 14, 2020 16:29
@bretg bretg self-assigned this Apr 16, 2020
@bretg
Copy link
Contributor

bretg commented Apr 16, 2020

@jdwieland8282 - I assume there will be updates to this after yesterday's conversation about adding 3rd party?

@@ -604,6 +605,70 @@ pbjs.setConfig({
});
{% endhighlight %}

#### SharedID

The SharedId UUID is a native Prebid identity service. SharedID has been designed to minimize its footprint on publishers pages, and optimized for the most common identity use case, interest based advertising. SharedID writes a simple 1st party ID in the publisher's domain. The first 48 bits represents the UUID’s birthdate, followed by 80 bits of random characters. Yes, a user will have a unique SharedID identity for each publisher they visit. However, since each UUID is available to all bidders who integrate with this identity service, we expect an open & transparent graph of users to emerge. Delegating this task to the community, and supporting it with this simple, efficient 1st party identity is beneficial to all parties participating in the RTB ecosystem.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The comma is not needed here:
SharedID has been designed to minimize its footprint on publishers pages, and optimized for the most common identity use case, interest based advertising.

Suggested minor edits:
Yes, a user will have a unique SharedID identity for each publisher they visit. However, since each UUID is available to all bidders who integrate with this identity service, we expect an open & transparent graph of users to emerge. Delegating this task to the community, and supporting it with this simple, efficient 1st party identity is beneficial to all parties participating in the RTB ecosystem.

should be:

A user will have a unique SharedID identity for each publisher they visit. Since each UUID is available to all bidders who integrate with this identity service we expect an open & transparent graph of users to emerge. Delegating this task to the community, and supporting it with this simple, efficient 1st party identity is beneficial to all parties participating in the RTB ecosystem.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry, didn't notice this, suggested edit:

SharedID has been designed to minimize its footprint on publishers pages and is optimized for the most common identity use case, interest-based advertising.

Copy link
Collaborator

@MartianTribe MartianTribe left a comment

Choose a reason for hiding this comment

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

Some minor edits.

@smenzer
Copy link
Collaborator

smenzer commented Apr 20, 2020

I'm curious...how is this any different than pubcommon? I also haven't seen any PRs in https://github.com/prebid/Prebid.js/pulls for this, I assume that's coming separately?

@@ -67,7 +68,7 @@ of sub-objects. The table below has the options that are common across ID system
{: .table .table-bordered .table-striped }
| Param under userSync.userIds[] | Scope | Type | Description | Example |
| --- | --- | --- | --- | --- |
| name | Required | String | May be: `"britepoolId"`, `"criteo"`, `"digitrust"`, `"id5id"`, `identityLink`, `"liveIntentId"`, `"parrableId"`, `"netId"`, `"pubCommonId"`, or `"unifiedId"` | `"unifiedId"` |
| name | Required | String | May be: `"britepoolId"`, `"criteo"`, `"digitrust"`, `"id5id"`, `identityLink`, `"liveIntentId"`, `"parrableId"`, `"netId"`, `"pubCommonId"`, `"SharedID"`, or `"unifiedId"` | `"unifiedId"` |
Copy link
Collaborator

Choose a reason for hiding this comment

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

shouldn't this be "sharedId" (lowercase "s" and "d")?

Suggested change
| name | Required | String | May be: `"britepoolId"`, `"criteo"`, `"digitrust"`, `"id5id"`, `identityLink`, `"liveIntentId"`, `"parrableId"`, `"netId"`, `"pubCommonId"`, `"SharedID"`, or `"unifiedId"` | `"unifiedId"` |
| name | Required | String | May be: `"britepoolId"`, `"criteo"`, `"digitrust"`, `"id5id"`, `identityLink`, `"liveIntentId"`, `"parrableId"`, `"netId"`, `"pubCommonId"`, `"sharedId"`, or `"unifiedId"` | `"unifiedId"` |

Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

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

doc needs to define how it's represented in bidRequest.userIds and in user.ext.eids.

@jdwieland8282
Copy link
Member Author

closing in place of #2031

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants