Skip to content

Commit 97ea0df

Browse files
committed
convert bidders: axonix
1 parent 0f5d97b commit 97ea0df

11 files changed

+38
-46
lines changed

modules/appierBidAdapter.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,7 @@ export const spec = {
4444
const payload = {
4545
'bids': bidRequests,
4646
// TODO: please do not pass internal data structures over to the network
47-
'refererInfo': {
48-
...bidderRequest.refererInfo,
49-
referer: bidderRequest.refererInfo.topmostLocation,
50-
},
47+
'refererInfo': bidderRequest.refererInfo.legacy,
5148
'version': ADAPTER_VERSION
5249
};
5350
return [{

modules/asealBidAdapter.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,8 @@ export const spec = {
5959

6060
const data = {
6161
bids: validBidRequests,
62-
refererInfo: bidderRequest.refererInfo,
62+
// TODO: please do not pass internal data structures over to the network
63+
refererInfo: bidderRequest.refererInfo?.legacy,
6364
device: {
6465
webSessionId: getTrekWebSessionId(),
6566
},

modules/asoBidAdapter.js

Lines changed: 7 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import {registerBidder} from '../src/adapters/bidderFactory.js';
33
import {config} from '../src/config.js';
44
import {BANNER, VIDEO} from '../src/mediaTypes.js';
55
import {Renderer} from '../src/Renderer.js';
6+
import {parseDomain} from '../src/refererDetection.js';
67

78
const BIDDER_CODE = 'aso';
89
const DEFAULT_SERVER_URL = 'https://srv.aso1.net';
@@ -167,28 +168,13 @@ function createRenderer(bid, url) {
167168
}
168169

169170
function getUrlsInfo(bidderRequest) {
170-
let page = '';
171-
let referrer = '';
172-
173-
const {refererInfo} = bidderRequest;
174-
175-
if (inIframe()) {
176-
page = refererInfo.referer;
177-
} else {
178-
const w = getWindowTop();
179-
page = w.location.href;
180-
referrer = w.document.referrer || '';
181-
}
182-
183-
page = config.getConfig('pageUrl') || page;
184-
const url = parseUrl(page);
185-
const domain = url.hostname;
186-
171+
const {page, domain, ref} = bidderRequest.refererInfo;
187172
return {
188-
domain,
189-
page,
190-
referrer
191-
};
173+
// TODO: do the fallbacks make sense here?
174+
page: page || bidderRequest.refererInfo?.topmostLocation,
175+
referrer: ref || '',
176+
domain: domain || parseDomain(bidderRequest?.refererInfo?.topmostLocation)
177+
}
192178
}
193179

194180
function getSize(paramSizes) {

modules/astraoneBidAdapter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ export const spec = {
9999
*/
100100
buildRequests(validBidRequests, bidderRequest) {
101101
const payload = {
102-
url: bidderRequest.refererInfo.referer,
102+
url: bidderRequest.refererInfo.page,
103103
cmp: !!bidderRequest.gdprConsent,
104104
bidRequests: buildBidRequests(validBidRequests)
105105
};

modules/audiencerunBidAdapter.js

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,7 @@ function getPageReferer() {
7171
* @return {string}
7272
*/
7373
function getPageUrl(bidderRequest) {
74-
return (
75-
config.getConfig('pageUrl') ||
76-
deepAccess(bidderRequest, 'refererInfo.referer') ||
77-
getPageReferer() ||
78-
null
79-
);
74+
return bidderRequest?.refererInfo?.page
8075
}
8176

8277
export const spec = {
@@ -127,10 +122,12 @@ export const spec = {
127122

128123
const payload = {
129124
libVersion: this.version,
130-
pageUrl: config.getConfig('pageUrl'),
125+
pageUrl: bidderRequest?.refererInfo?.page,
126+
// TODO: does it make sense to find a half-way referer? what should these parameters pick
131127
pageReferer: getPageReferer(),
132-
referer: deepAccess(bidderRequest, 'refererInfo.referer'),
133-
refererInfo: deepAccess(bidderRequest, 'refererInfo'),
128+
referer: deepAccess(bidderRequest, 'refererInfo.topmostLocation'),
129+
// TODO: please do not send internal data structures over the network
130+
refererInfo: deepAccess(bidderRequest, 'refererInfo.legacy'),
134131
currencyCode: config.getConfig('currency.adServerCurrency'),
135132
timeout: config.getConfig('bidderTimeout'),
136133
bids,

modules/automatadBidAdapter.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,9 @@ export const spec = {
6161
imp: impressions,
6262
site: {
6363
id: siteId,
64-
domain: window.location.hostname,
65-
page: window.location.href,
66-
ref: bidderRequest.refererInfo ? bidderRequest.refererInfo.referer || null : null,
64+
domain: bidderRequest.refererInfo?.domain,
65+
page: bidderRequest.refererInfo?.page,
66+
ref: bidderRequest.refererInfo?.ref
6767
},
6868
}
6969

modules/axonixBidAdapter.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,11 @@ function getBidFloor(bidRequest) {
2525
}
2626

2727
function getPageUrl(bidRequest, bidderRequest) {
28-
let pageUrl = config.getConfig('pageUrl');
29-
28+
let pageUrl;
3029
if (bidRequest.params.referrer) {
3130
pageUrl = bidRequest.params.referrer;
32-
} else if (!pageUrl) {
33-
pageUrl = bidderRequest.refererInfo.referer;
31+
} else {
32+
pageUrl = bidderRequest.refererInfo.page;
3433
}
3534

3635
return bidRequest.params.secure ? pageUrl.replace(/^http:/i, 'https:') : pageUrl;

src/refererDetection.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,16 @@ export function detectReferer(win) {
240240
page,
241241
domain: parseDomain(page) || null,
242242
ref: ref || null,
243+
// TODO: the "legacy" refererInfo object is provided here, for now, to accomodate
244+
// adapters that decided to just sent it verbatim to their backend.
245+
legacy: {
246+
reachedTop,
247+
isAmp: valuesFromAmp,
248+
numIframes: level - 1,
249+
stack,
250+
referer: bestLocation || null,
251+
canonicalUrl: bestCanonicalUrl || null,
252+
}
243253
};
244254
}
245255

test/spec/modules/asoBidAdapter_spec.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ describe('Adserver.Online bidding adapter', function () {
6262
refererInfo: {
6363
numIframes: 0,
6464
reachedTop: true,
65-
referer: 'https://example.com'
65+
page: 'https://example.com',
66+
domain: 'example.com'
6667
}
6768
};
6869

test/spec/modules/audiencerunBidAdapter_spec.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,8 @@ describe('AudienceRun bid adapter tests', function () {
114114
},
115115
refererInfo: {
116116
canonicalUrl: undefined,
117-
referer: 'https://example.com',
117+
page: 'https://example.com',
118+
topmostLocation: 'https://example.com',
118119
numIframes: 0,
119120
reachedTop: true,
120121
},

test/spec/modules/axonixBidAdapter_spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ describe('AxonixBidAdapter', function () {
6565
gdprApplies: true
6666
},
6767
refererInfo: {
68-
referer: 'https://www.prebid.org',
68+
page: 'https://www.prebid.org',
6969
canonicalUrl: 'https://www.prebid.org/the/link/to/the/page'
7070
}
7171
};

0 commit comments

Comments
 (0)