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.

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

Pubmatic: fix issue where using an adUnit outstream renderer throws an error #6152

Merged
merged 71 commits into from
Jan 28, 2021
Merged
Changes from 69 commits
Commits
Show all changes
71 commits
Select commit Hold shift + click to select a range
7e5986c
Merge pull request #1 from prebid/master
pm-shashank-jain Nov 27, 2018
d1146d7
Merge pull request #2 from prebid/master
pm-shashank-jain Nov 29, 2018
609ec16
changes to support native in pubmaticbid adapter
pm-shashank-jain Dec 5, 2018
792b854
Removed port from endpoint
pm-shashank-jain Dec 5, 2018
b6740d0
Removed protocol from endpoint
pm-shashank-jain Dec 5, 2018
0569f41
Formatting
pm-shashank-jain Dec 5, 2018
fd6bf03
Fix request payload
pm-shashank-jain Dec 6, 2018
bd14420
Updated test case
pm-shashank-jain Dec 6, 2018
65e2f1e
Changed request and response as per ortb spec
pm-shashank-jain Dec 7, 2018
13c87de
Change in request and response
pm-shashank-jain Dec 10, 2018
d9f3562
Removed comments and extra code
pm-shashank-jain Dec 10, 2018
adbc774
Code Review comments
pm-shashank-jain Dec 11, 2018
21fe846
Code Review Comments and Test cases for request and response
pm-shashank-jain Dec 13, 2018
9faff35
Removed data type as all data asset types are handled
pm-shashank-jain Dec 14, 2018
78cd16d
Code Review Changes
pm-shashank-jain Dec 14, 2018
4a40e1e
Code Review Comments
pm-shashank-jain Dec 17, 2018
a218d43
Supporting both banner and native and sending 0x0 in case of native
pm-shashank-jain Dec 18, 2018
f65b3fb
Bug Fixes
pm-shashank-jain Dec 21, 2018
a717fbc
Bug response not processed by prebid
pm-shashank-jain Dec 21, 2018
af2503d
Change warning message
pm-shashank-jain Dec 21, 2018
21277c1
Fixed typo
pm-shashank-jain Dec 21, 2018
53ac4fe
Merge pull request #4 from prebid/master
pm-shashank-jain Dec 21, 2018
ac1919f
Merge branch 'master' of github.com:pm-shashank-jain/Prebid.js into n…
pm-shashank-jain Dec 21, 2018
1de1623
Do not send request in case of invalid native bid
pm-shashank-jain Dec 24, 2018
6b059b7
Do not send request in case of invalid native requests
pm-shashank-jain Dec 24, 2018
9c80c46
objects converted to strings in log for debug purposes
pm-shashank-jain Dec 24, 2018
e0f8763
Fixed logic to check for required parmas
pm-shashank-jain Dec 24, 2018
c4ee495
Fixed typo for stringify
pm-shashank-jain Dec 24, 2018
d4abb65
Merge pull request #3 from pm-shashank-jain/nativesupport
pm-shashank-jain Dec 26, 2018
f143cee
documentation for native
pm-shashank-jain Dec 26, 2018
06a6e34
Merge pull request #5 from pm-shashank-jain/nativesupport
pm-shashank-jain Dec 26, 2018
84a9c02
Review comments from Prebid
pm-shashank-jain Jan 8, 2019
e3d068e
Typo
pm-shashank-jain Jan 8, 2019
468c3ab
Typo
pm-shashank-jain Jan 8, 2019
11fce22
Updated pub id for native
pm-shashank-jain Jan 10, 2019
b23e2dd
Code Review
pm-shashank-jain Jan 15, 2019
6b8c31b
Merge pull request #6 from pm-shashank-jain/nativesupport
pm-shashank-jain Jan 15, 2019
2f73d99
Merge branch 'master' of github.com:pm-shashank-jain/Prebid.js
pm-shashank-jain Aug 21, 2019
27e56d6
Latest Code
pm-shashank-jain Aug 21, 2019
9c0772b
Merge pull request #8 from prebid/master
pm-shashank-jain Aug 30, 2019
909f526
Support for pubid
pm-shashank-jain Aug 30, 2019
7fe0f35
Merge branch 'master' of github.com:pm-shashank-jain/Prebid.js
pm-shashank-jain Aug 30, 2019
53a7ddf
Test Cases for PubCommonId in PubMatic adapter
pm-shashank-jain Sep 3, 2019
f168aea
Delete yarn.lock
pm-shashank-jain Sep 3, 2019
e9db7f3
Rename adaptermanager.js to adapterManager.js
pm-shashank-jain Sep 3, 2019
e753a97
Rename yieldNexusBidAdapter.js to yieldnexusBidAdapter.js
pm-shashank-jain Sep 3, 2019
68b2e65
Rename yieldNexusBidAdapter.md to yieldnexusBidAdapter.md
pm-shashank-jain Sep 3, 2019
483cddf
Rename yieldNexusBidAdapter_spec.js to yieldnexusBidAdapter_spec.yiel…
pm-shashank-jain Sep 3, 2019
149875b
Rename yieldnexusBidAdapter_spec.yieldnexusBidAdaptera to yieldnexusB…
pm-shashank-jain Sep 3, 2019
8d89338
took latest
pm-shashank-jain Dec 27, 2019
e5b4722
Merge branch 'prebid-master'
pm-shashank-jain Dec 27, 2019
7a0cace
Merge remote-tracking branch 'upstream/master'
pm-shashank-jain Jun 12, 2020
a1d5256
bluebillywig outstream player support in pubmatic adapter
pm-shashank-jain Jun 23, 2020
6e4fa37
Merge https://github.com/prebid/Prebid.js
pm-shashank-jain Jul 2, 2020
fb6056c
Merge branch 'master' of github.com:pm-shashank-jain/Prebid.js into o…
pm-shashank-jain Jul 2, 2020
5d5e4eb
Merge https://github.com/prebid/Prebid.js into outstream_renderer
pm-shashank-jain Jul 15, 2020
825242c
Merge https://github.com/prebid/Prebid.js into outstream_renderer
pm-shashank-jain Jul 17, 2020
4681528
Merge https://github.com/prebid/Prebid.js into outstream_renderer
pm-shashank-jain Jul 21, 2020
1ee6583
removed pubcommon id test cases
pm-shashank-jain Jul 21, 2020
260f7f7
Merge https://github.com/prebid/Prebid.js
pm-shashank-jain Jul 21, 2020
cc7c65c
BBW Renderer
pm-shashank-jain Jul 21, 2020
8a2e508
Merge branch 'master' of github.com:pm-shashank-jain/Prebid.js into o…
pm-shashank-jain Jul 21, 2020
3362acd
Merge https://github.com/prebid/Prebid.js
pm-shashank-jain Jul 30, 2020
e72f47f
Merge pull request #11 from pm-shashank-jain/outstream_renderer
pm-shashank-jain Jul 30, 2020
251398e
Merge branch 'master' of https://github.com/prebid/Prebid.js
pm-shashank-jain Aug 12, 2020
efdf7a1
Took latest prebid
pm-shashank-jain Dec 16, 2020
936f0fe
Merge branch 'master' of https://github.com/prebid/Prebid.js
pm-shashank-jain Dec 28, 2020
af471f8
Pubmatic should work with provided renderer
pm-shashank-jain Dec 28, 2020
7048c86
Merge pull request #12 from pm-shashank-jain/outstream_bug_fix
pm-shashank-jain Dec 28, 2020
b75de91
Merge branch 'master' of https://github.com/prebid/Prebid.js
pm-shashank-jain Jan 27, 2021
92ee235
Review Comments
pm-shashank-jain Jan 27, 2021
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/pubmaticBidAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -861,8 +861,8 @@ export const spec = {
utils.logError(`${LOG_WARN_PREFIX}: no context specified in bid. Rejecting bid: `, bid);
return false;
}
if (bid.mediaTypes[VIDEO].context === 'outstream' && !utils.isStr(bid.params.outstreamAU)) {
utils.logError(`${LOG_WARN_PREFIX}: for "outstream" bids outstreamAU is required. Rejecting bid: `, bid);
if (bid.mediaTypes[VIDEO].context === 'outstream' && !utils.isStr(bid.params.outstreamAU) && !bid.hasOwnProperty('renderer')) {
ncolletti marked this conversation as resolved.
Show resolved Hide resolved
utils.logError(`${LOG_WARN_PREFIX}: for "outstream" bids either outstreamAU parameter must be provided or ad unit supplied renderer is required. Rejecting bid: `, bid);
return false;
}
} else {
Expand Down