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

IX Adapter - Increase banner TTL to 300s (legacy 2.44.x) #4958

Closed
wants to merge 1 commit into from
Closed

IX Adapter - Increase banner TTL to 300s (legacy 2.44.x) #4958

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
2 changes: 1 addition & 1 deletion modules/ixBidAdapter.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const SUPPORTED_AD_TYPES = [BANNER, VIDEO];
const BANNER_ENDPOINT_VERSION = 7.2;
const VIDEO_ENDPOINT_VERSION = 8.1;
const CENT_TO_DOLLAR_FACTOR = 100;
const BANNER_TIME_TO_LIVE = 35;
const BANNER_TIME_TO_LIVE = 300;
const VIDEO_TIME_TO_LIVE = 3600; // 1hr
const NET_REVENUE = true;
const PRICE_TO_DOLLAR_FACTOR = {
Expand Down
2 changes: 1 addition & 1 deletion modules/ixBidAdapter.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -365,7 +365,7 @@ to `'ix'` across all ad units that bids are being requested for does not exceed

### Time-To-Live (TTL)

All bids received from IX have a TTL of 35 seconds, after which time they become
Banner bids from IX have a TTL of 300 seconds while video bids have a TTL of 1 hour, after which time they become
invalid.

If an invalid bid wins, and its associated ad is rendered, it will not count
Expand Down
8 changes: 4 additions & 4 deletions test/spec/modules/ixBidAdapter_spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -807,7 +807,7 @@ describe('IndexexchangeAdapter', function () {
mediaType: 'banner',
ad: '<a target="_blank" href="http://www.indexexchange.com"></a>',
currency: 'USD',
ttl: 35,
ttl: 300,
netRevenue: true,
dealId: undefined,
meta: {
Expand All @@ -834,7 +834,7 @@ describe('IndexexchangeAdapter', function () {
mediaType: 'banner',
ad: '<a target="_blank" href="http://www.indexexchange.com"></a>',
currency: 'USD',
ttl: 35,
ttl: 300,
netRevenue: true,
dealId: undefined,
meta: {
Expand All @@ -860,7 +860,7 @@ describe('IndexexchangeAdapter', function () {
mediaType: 'banner',
ad: '<a target="_blank" href="http://www.indexexchange.com"></a>',
currency: 'JPY',
ttl: 35,
ttl: 300,
netRevenue: true,
dealId: undefined,
meta: {
Expand All @@ -887,7 +887,7 @@ describe('IndexexchangeAdapter', function () {
mediaType: 'banner',
ad: '<a target="_blank" href="http://www.indexexchange.com"></a>',
currency: 'USD',
ttl: 35,
ttl: 300,
netRevenue: true,
dealId: 'deal',
meta: {
Expand Down