Skip to content

Commit

Permalink
Kargo Bid Adapter: Fix issue with double-array for advertiserDomains (p…
Browse files Browse the repository at this point in the history
…rebid#7910)

* Fix issue with double-array for advertiserDomains

* Updated test to reflect correct advertiserDomains rray

* clickUrl to pull first advertiserDomain
  • Loading branch information
jsadwith authored Jan 6, 2022
1 parent 355f45f commit 30c6f40
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions modules/kargoBidAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,8 @@ export const spec = {
let meta;
if (adUnit.metadata && adUnit.metadata.landingPageDomain) {
meta = {
clickUrl: adUnit.metadata.landingPageDomain,
advertiserDomains: [adUnit.metadata.landingPageDomain]
clickUrl: adUnit.metadata.landingPageDomain[0],
advertiserDomains: adUnit.metadata.landingPageDomain
};
}
bidResponses.push({
Expand Down
2 changes: 1 addition & 1 deletion test/spec/modules/kargoBidAdapter_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -484,7 +484,7 @@ describe('kargo adapter tests', function () {
height: 250,
targetingCustom: 'dmpmptest1234',
metadata: {
landingPageDomain: 'https://foobar.com'
landingPageDomain: ['https://foobar.com']
}
},
3: {
Expand Down

0 comments on commit 30c6f40

Please sign in to comment.