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

Relaido Bid Adapter: change to get the renderer URL for each response #8085

Merged
merged 17 commits into from
Feb 18, 2022
Merged

Relaido Bid Adapter: change to get the renderer URL for each response #8085

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
1d0240f
add relaido adapter
r-ishigami Apr 9, 2020
9be8275
remove event listener
r-ishigami Apr 9, 2020
d63a27a
fixed UserSyncs and e.data
r-ishigami Apr 10, 2020
8c7f8a0
fix conflicts
r-ishigami May 20, 2020
6cb31b6
fix conflicts
r-ishigami May 20, 2020
114d6d2
Merge remote-tracking branch 'root_branch/master'
r-ishigami Jul 6, 2020
e0092cc
Merge remote-tracking branch 'root_branch/master' into master
Oct 9, 2020
dd0b7d7
Merge remote-tracking branch 'root_branch/master'
Oct 29, 2020
975f838
Merge remote-tracking branch 'root_branch/master'
Nov 18, 2020
0040be7
Merge remote-tracking branch 'root_branch/master'
May 17, 2021
5e086c4
Merge remote-tracking branch 'root_branch/master'
Jun 2, 2021
0572f86
Merge remote-tracking branch 'root_branch/master'
Jun 2, 2021
56da7ec
Merge remote-tracking branch 'root_branch/master'
Jun 28, 2021
11cf5ef
Merge remote-tracking branch 'root_branch/master'
Sep 16, 2021
abb742b
Merge remote-tracking branch 'root_branch/master'
Oct 1, 2021
0235ffa
Merge remote-tracking branch 'root_branch/master' into master
nmaeura Feb 16, 2022
bc1d6e4
supports for the case where playerUrl is defined inside ads
relaido Feb 18, 2022
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
4 changes: 2 additions & 2 deletions modules/relaidoBidAdapter.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,16 +121,16 @@ function interpretResponse(serverResponse, bidRequest) {
return [];
}

const playerUrl = bidRequest.player || body.playerUrl;

for (const res of body.ads) {
const playerUrl = res.playerUrl || bidRequest.player || body.playerUrl;
let bidResponse = {
requestId: res.bidId,
width: res.width,
height: res.height,
cpm: res.price,
currency: res.currency,
creativeId: res.creativeId,
playerUrl: playerUrl,
dealId: body.dealId || '',
ttl: body.ttl || DEFAULT_TTL,
netRevenue: true,
Expand Down
19 changes: 19 additions & 0 deletions test/spec/modules/relaidoBidAdapter_spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -303,6 +303,7 @@ describe('RelaidoAdapter', function () {
expect(response.currency).to.equal(serverResponse.body.ads[0].currency);
expect(response.creativeId).to.equal(serverResponse.body.ads[0].creativeId);
expect(response.vastXml).to.equal(serverResponse.body.ads[0].vast);
expect(response.playerUrl).to.equal(serverResponse.body.playerUrl);
expect(response.meta.advertiserDomains).to.equal(serverResponse.body.ads[0].adomain);
expect(response.meta.mediaType).to.equal(VIDEO);
expect(response.ad).to.be.undefined;
Expand All @@ -320,11 +321,29 @@ describe('RelaidoAdapter', function () {
expect(response.currency).to.equal(serverResponse.body.ads[0].currency);
expect(response.creativeId).to.equal(serverResponse.body.ads[0].creativeId);
expect(response.vastXml).to.be.undefined;
expect(response.playerUrl).to.equal(serverResponse.body.playerUrl);
expect(response.ad).to.include(`<div id="rop-prebid">`);
expect(response.ad).to.include(`<script src="https://relaido/player.js"></script>`);
expect(response.ad).to.include(`window.RelaidoPlayer.renderAd`);
});

it('should build bid response by video and playerUrl in ads', function () {
serverResponse.body.ads[0].playerUrl = 'https://relaido/player-customized.js';
const bidResponses = spec.interpretResponse(serverResponse, serverRequest);
expect(bidResponses).to.have.lengthOf(1);
const response = bidResponses[0];
expect(response.playerUrl).to.equal(serverResponse.body.ads[0].playerUrl);
});

it('should build bid response by banner and playerUrl in ads', function () {
serverResponse.body.ads[0].playerUrl = 'https://relaido/player-customized.js';
serverResponse.body.ads[0].mediaType = 'banner';
const bidResponses = spec.interpretResponse(serverResponse, serverRequest);
expect(bidResponses).to.have.lengthOf(1);
const response = bidResponses[0];
expect(response.playerUrl).to.equal(serverResponse.body.ads[0].playerUrl);
});

it('should not build bid response', function () {
serverResponse = {};
const bidResponses = spec.interpretResponse(serverResponse, serverRequest);
Expand Down