Skip to content

Commit 0170539

Browse files
committed
convert bidders: adgeneration
1 parent f849022 commit 0170539

9 files changed

+55
-52
lines changed

modules/adbookpspBidAdapter.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,9 +123,9 @@ function buildRequest(validBidRequests, bidderRequest) {
123123
id: bidderRequest.bidderRequestId,
124124
tmax: bidderRequest.timeout,
125125
site: {
126-
domain: window.location.hostname,
127-
page: window.location.href,
128-
ref: bidderRequest.refererInfo.referer,
126+
domain: bidderRequest.refererInfo.domain,
127+
page: bidderRequest.refererInfo.page,
128+
ref: bidderRequest.refererInfo.ref,
129129
},
130130
source: buildSource(validBidRequests, bidderRequest),
131131
device: buildDevice(),

modules/addefendBidAdapter.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@ export const spec = {
2121
pageId: false,
2222
gdpr_applies: bidderRequest.gdprConsent && bidderRequest.gdprConsent.gdprApplies ? bidderRequest.gdprConsent.gdprApplies : 'true',
2323
gdpr_consent: bidderRequest.gdprConsent && bidderRequest.gdprConsent.consentString ? bidderRequest.gdprConsent.consentString : '',
24-
referer: bidderRequest.refererInfo.referer,
24+
// TODO: is 'page' the correct item here?
25+
referer: bidderRequest.refererInfo.page,
2526
bids: [],
2627
};
2728

modules/adfBidAdapter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ export const spec = {
8181
}
8282

8383
if (!site.page) {
84-
site.page = bidderRequest.refererInfo.referer;
84+
site.page = bidderRequest.refererInfo.page;
8585
}
8686
}
8787

modules/adgenerationBidAdapter.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,8 @@ export const spec = {
4949
if (!validReq.mediaTypes || !validReq.mediaTypes.native) {
5050
data = tryAppendQueryString(data, 'imark', '1');
5151
}
52-
data = tryAppendQueryString(data, 'tp', bidderRequest.refererInfo.referer);
52+
// TODO: is 'page' the right value here?
53+
data = tryAppendQueryString(data, 'tp', bidderRequest.refererInfo.page);
5354
if (isIos()) {
5455
const hyperId = getHyperId(validReq);
5556
if (hyperId != null) {

test/spec/modules/adagioBidAdapter_spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ import { loadExternalScript } from '../../../src/adloader.js';
1616
import * as utils from '../../../src/utils.js';
1717
import { config } from '../../../src/config.js';
1818
import { NATIVE } from '../../../src/mediaTypes.js';
19-
import * as prebidGlobal from 'src/prebidGlobal.js';
2019
import { executeRenderer } from '../../../src/Renderer.js';
2120

2221
const BidRequestBuilder = function BidRequestBuilder(options) {

test/spec/modules/adbookpspBidAdapter_spec.js

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -951,7 +951,9 @@ const bidderRequest = {
951951
bidderRequestId: '999ccceeee11',
952952
timeout: 200,
953953
refererInfo: {
954-
referer: 'http://example-domain.com/foo',
954+
page: 'http://mock-page.com',
955+
domain: 'mock-page.com',
956+
ref: 'http://example-domain.com/foo',
955957
},
956958
gdprConsent: {
957959
gdprApplies: 1,
@@ -999,8 +1001,8 @@ const bannerExchangeRequest = {
9991001
},
10001002
},
10011003
site: {
1002-
domain: location.hostname,
1003-
page: location.href,
1004+
domain: 'mock-page.com',
1005+
page: 'http://mock-page.com',
10041006
ref: 'http://example-domain.com/foo',
10051007
},
10061008
source: {
@@ -1089,8 +1091,8 @@ const videoExchangeRequest = {
10891091
},
10901092
},
10911093
site: {
1092-
domain: location.hostname,
1093-
page: location.href,
1094+
domain: 'mock-page.com',
1095+
page: 'http://mock-page.com',
10941096
ref: 'http://example-domain.com/foo',
10951097
},
10961098
source: {
@@ -1171,8 +1173,8 @@ const mixedExchangeRequest = {
11711173
},
11721174
},
11731175
site: {
1174-
domain: location.hostname,
1175-
page: location.href,
1176+
domain: 'mock-page.com',
1177+
page: 'http://mock-page.com',
11761178
ref: 'http://example-domain.com/foo',
11771179
},
11781180
source: {

test/spec/modules/addefendBidAdapter_spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ describe('addefendBidAdapter', () => {
2525
return spec.buildRequests(buildRequest, {
2626
...bidderRequest || {},
2727
refererInfo: {
28-
referer: 'https://referer.example.com'
28+
page: 'https://referer.example.com'
2929
}
3030
})[0];
3131
};

test/spec/modules/adfBidAdapter_spec.js

Lines changed: 36 additions & 36 deletions
Large diffs are not rendered by default.

test/spec/modules/adgenerationBidAdapter_spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ describe('AdgenerationAdapter', function () {
105105
];
106106
const bidderRequest = {
107107
refererInfo: {
108-
referer: 'https://example.com'
108+
page: 'https://example.com'
109109
}
110110
};
111111
const data = {

0 commit comments

Comments
 (0)