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.

Sign up for GitHub

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

Jump to bottom

AppNexus: Added info for generate_ad_pod_id #2836

Merged
merged 2 commits into from
Jun 10, 2021
Merged

AppNexus: Added info for generate_ad_pod_id #2836

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c2682ca
Added info for generate_ad_pod_id
Apr 8, 2021
d0fe024
Added info for generate_ad_pod_id
Apr 15, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions dev-docs/bidders/appnexus.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ All AppNexus placements included in a single call to `requestBids` must belong t
| `pubClick` | optional | Specifies a publisher-supplied URL for third-party click tracking. This is just a placeholder into which the publisher can insert their own click tracker. This parameter should be used for an unencoded tracker. This parameter is expected to be the last parameter in the URL. Please note that the click tracker placed in this parameter will only fire if the creative winning the auction is using AppNexus click tracking properly. | `'http://click.adserver.com/'` | `string` |
| `extInvCode` | optional | Specifies predefined value passed on the query string that can be used in reporting. The value must be entered into the system before it is logged. | `'10039'` | `string` |
| `externalImpId` | optional | Specifies the unique identifier of an externally generated auction. | `'bacbab02626452b097f6030b3c89ac05'` | `string` |
| `generate_ad_pod_id`| optional | Signal to AppNexus to split impressions by ad pod and add unique ad pod id to each request. Specific to long form video endpoint only. | `true` | `boolean` |
Copy link

Choose a reason for hiding this comment

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

May want to change it to Intended for long form video endpoint only.. There doesn't seem to be anything to prevent it being used in non-long form video, but also does not make sense in this context either. Perhaps someone will come up with a smart twist that uses this on the main openrtb endpoint.

Copy link
Contributor

Choose a reason for hiding this comment

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

It will be ignored for non-long form video because the code branch is only activated for /video traffic. I like the idea of adding the long form video endpoint only note. I'd also think it's important to note this is only supported by PBS and not PBJS (there's no need on the PBJS side as far as I'm aware).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modified.


<a name="appnexus-video-object" />

Expand Down