refactor using ORTB conversion library #29

ahmadlob · 2024-01-02T09:13:34Z

Type of change

Bugfix
Feature
New bidder adapter
Code style update (formatting, local variables)
Refactoring (no functional changes, no api changes)
Build related changes
CI related changes
Does this change affect user-facing APIs or examples documented on http://prebid.org?
Other

Description of change

Other information

* identityLinkIdSystem - liveramp - include gpp in 3p envelope retrieval * identityLinkIdSystem - liveramp - adjust test description

* tmax * tmax * tmax * tmax

* adxcgBidAdapter: fix convertTypes util import * adxcgBidAdapter: remove previous import

* new boldwin bid adapter * fix * Restarting ci / circleci * changes * fixed conflicts * added endpointId param * kick off CircleCI tests * gpp * update sync url --------- Co-authored-by: Aiholkin <[email protected]> Co-authored-by: Vladislav Isaiko <[email protected]> Co-authored-by: Mykhailo Yaremchuk <[email protected]> Co-authored-by: Chris Huie <[email protected]>

Co-authored-by: federico <[email protected]>

* cookie deprecation label * reverted unneeded changes made during development * addressed feedback * removed unnecessary else statement * added comment for FIRST_PARTY_GVLID param * ensure promise resolves when label is falsey * addressed feedback * removed async/await from getCookieDeprecationLabel * updating tests * more changes * addressed feedback * Small improvements --------- Co-authored-by: Demetrio Girardi <[email protected]>

…and no conversion is done (#10736) * Currency module: Don't block 'responsesReady' when rates are present and no conversion is needed * Add test case for currency unnecessarily blocking auctions * Update currency_spec.js --------- Co-authored-by: Demetrio Girardi <[email protected]>

…n build (#10749) * New adapter: concepx * Syntax change * Revert syntax change * Defensive check for response from bidder server * Add better validation for the request * Merge branch 'master' of https://github.com/prebid/Prebid.js * Don't append url on every buildrequest

Co-authored-by: v.raybaud <[email protected]>

* Remove use of local storage As requested, we remove the use of local storage. prebid/Prebid.js#8689 * Update impactifyBidAdapter.js * Add differents mediatypes to Impactify bidder * Add differents mediatypes to Impactify bidder * Add differents mediatypes to Impactify bidder * Add format parameter for banner * add getFloor * add getFloor * add getFloor * add parsing of local storage * delete unused var * fix spacing with import * Add local storage key management * Adjustments * Fix eids object * Fix eids object * Fix eids object * Fix tests * Remove check on the request banner size parameter * Few adjustments --------- Co-authored-by: Thomas De Stefano <[email protected]> Co-authored-by: Pang Ronnie <[email protected]>

* Unruly Protected Audience Support * Unruly Conditional Protected Audience Support * Unruly map bidIds to AuctionConfigs * Unruly: Update Protected Audience origin and descision logic URL. * Unruly: Fix lint error --------- Co-authored-by: Brian Haines <[email protected]>

…#10762) * add relaido adapter * remove event listener * fixed UserSyncs and e.data * fix conflicts * add response placementID * add bidrequest some param --------- Co-authored-by: ishigami_shingo <[email protected]> Co-authored-by: cmertv-sishigami <[email protected]> Co-authored-by: t_bun <[email protected]> Co-authored-by: n.maeura <[email protected]>

* Missena Bid Adapter: add capping support * Switch to safeJSONParse * Update tests to cover capping

- invalid ad units are put into a voidAuIds array from the ad server. - this stores those ad units into local storage and prevents calling to those ad units. - also handles any values from metaData from the adserver response.

* TPMN Support Video * add video param plcmt * update ortb bcat,badv,bapp * instream video is sound on default, outstream video sound off default * remove VIDEO_ORTB_PARAMS, BANNER_ORTB_PARAMS * fixed incorrect placement default format & update example * update test case (Your tests failed on CircleCI) * Revert "remove VIDEO_ORTB_PARAMS, BANNER_ORTB_PARAMS" This reverts commit 2699115a76ed891ec7f911ad924206b399790ad9. * Remove unnecessary code to meet guidelines. * Remove unnecessary code to meet guidelines. * fix Build fail * delete logging code * Remove OptionsObject(Credentials information needs to be sent.) --------- Co-authored-by: changjun <[email protected]> Co-authored-by: yeongjaeju <[email protected]>

* Add dynamicAdBoost Module * Add dynamicAdBoost spec file * dynamicAdBoost: fix ES Lint * dynamicAdBoost: ESLint fix * dynamicAdBoost: ESLint fix * dynamicAdBoost: fix ESLint * dynamicAdBoost: removed unused log warn * dynamicAdBoost: Fixed tests * dynamicAdBoost: export init() function * Delete dynamicAdBoost.md * Delete dynamicAdBoost.js * Delete dynamicAdBoost_spec.js * Rename dynamicAdBoost to dynamicAdBoostRtdProvider dynamicAdBoost is now submodule of the rtd * dynamicAdBoost tests * Added disconnect() and IO support check * dynamicAdBoostRtdProvider: markViewad added * dynamicAdBoostRtdProvider: markViewed added * dynamicAdBoostRtdProvider.js: observer fix * dynamicAdBoostRtdProvider.js: observer fix * dynamicAdBoostRtdProvider.js: ESLint fix * dynamicAdBoostRtdProvider.js: var to let * dynamicAdBoostRtdProvider.js: moved timers * dynamicAdBoostRtdProvider_spec.js: using clock * dynamicAdBoostRtdProvider_spec.js: fakeTimer * Update adloader.js Adding dynamicAdBoost module to approved list

* Flipp Bid Adapter: initial release * Added flippBidAdapter * OFF-372 Support DTX/Hero in flippBidAdapter (#2) * support creativeType * OFF-422 flippBidAdapter handle AdTypes --------- Co-authored-by: Jairo Panduro <[email protected]> * OFF-465 Add getUserKey logic to prebid.js adapter (#3) * Support cookie sync and uid * address pr feedback * remove redundant check * OFF-500 Support "startCompact" param for Prebid.JS #4 * set startCompact default value (#5) * fix docs * use client bidding endpoint * update unit testing endpoint * OFF-876 [Prebid Adapter] Check userKey for empty string (#6) * add more checks to userKey * update document * add uuid format statement * modify docs * fix network id --------- Co-authored-by: Jairo Panduro <[email protected]>

…in 3p call (#10778)

* Add EID translation for sovrn * Review comments: fix test

* Unruly Protected Audience Support * Unruly Conditional Protected Audience Support * Unruly map bidIds to AuctionConfigs * Unruly: Update Protected Audience origin and descision logic URL. * Unruly: Fix lint error * Unruly: report Prebid.js version * Unruly: Fix tests * Unruly: Remove Extra Comma * Unruly: Switch to strings to see if tests pass on Prebid.js --------- Co-authored-by: Brian Haines <[email protected]>

* new adapter - MgidX * add new required param host * Revert "add new required param host" This reverts commit 3bafe782a492d5d4b35476f8117260b08a237f2f. * refactoring generating request.gdpr object * fix test --------- Co-authored-by: Evgeny Nagorny <[email protected]> Co-authored-by: xmgiddev <>

…ecat`, `site.content.cat` and `site.content.language` (#10761) * added support for gpp consent string * changed test for nextMillenniumBidAdapter * added some tests * added site.pagecat, site.content.cat and site.content.language to request * lint fix * formated code * formated code * formated code * pachage-lock with prebid * pachage-lock with prebid * formatted code

* Yandex Bid Adapter: add support for topicsFpdModule * add test

ahmadlob · 2024-01-02T09:17:46Z

modules/taboolaBidAdapter.js

+const converter = ortbConverter({
+  context: {
+    netRevenue: true,
+    ttl: 60


check the value in viper

netRevenue and ttl are required properties of bid responses - provide a default for them,
default bidResponse.ttl (when not specified in ORTB response.seatbid[].bid[].exp)

the value in viper is different per exchange, but it looks default should be "300"
check in viper:
com.taboola.model.common.CommonConfigProvider#getIntegerProxy
@barRubi

ahmadlob · 2024-01-02T09:22:37Z

modules/taboolaBidAdapter.js

+    return imp;
+  },
+  request(buildRequest, imps, bidderRequest, context) {
+    const req = buildRequest(imps, bidderRequest, context);


call helper method "buildTaboolaRequest" for example

ahmadlob · 2024-01-02T09:23:01Z

modules/taboolaBidAdapter.js

-        pageType: ortb2?.ext?.data?.pageType || ortb2?.ext?.data?.section || bidRequest.params.pageType
-      }
-    };
+    data.id = bidderRequest.bidderRequestId;


all lines here related to the request within the new helper function "buildTaboolaRequest"

ahmadlob · 2024-01-02T09:27:01Z

modules/taboolaBidAdapter.js

-function getImps(validBidRequests) {
-  return validBidRequests.map((bid, id) => {
+function getImps(bidRequest, imp) {
+  return bidRequest.map((bid, id) => {


bidRequest is a single element

ahmadlob · 2024-01-02T09:27:32Z

modules/taboolaBidAdapter.js

+  imp(buildImp, bidRequest, context) {
+    let imp = buildImp(bidRequest, context);
+    const imps = getImps([bidRequest], imp);
+    deepSetValue(bidRequest, 'imp', imps);


no need for this line

ahmadlob · 2024-01-02T09:28:54Z

modules/taboolaBidAdapter.js

-      banner: getBanners(bid, position),
-      tagid: tagId
-    }
+    deepSetValue(imp, 'banner', getBanners(bid, position));


deepSetValue is for inner object

ahmadlob · 2024-01-02T09:32:19Z

modules/taboolaBidAdapter.js

      return [];
    }

-    const {bidResponses, cur: currency} = getBidResponses(serverResponse);
+    if (!serverResponse || !serverResponse.body) {


null check for:
!serverResponse.body.seatbid

ahmadlob · 2024-01-02T09:33:58Z

test/spec/modules/taboolaBidAdapter_spec.js

-          }
-        ]
-      }
+      const multiRequest = spec.buildRequests([defaultBidRequest, defaultBidRequest], bidderRequest);


adding new test checking that the convertor passed any not specific new field into imp/req

* Initial implementation of kulturemedia bid adapter * Changing outstream to instream * Enriching md file with test examples * Changing nId to networkId * Cleaning up md file * Submitting rebranded dxkultureBidAdapter * dxkultureBidAdapter - Improve UserSyncs * Include gdpr/usp params in iframe usersync url * Add gdpr/usp data to iframe usync urls * Cleaning up testing html file * Adding outstream support * Updating exchange endpoint * Resolve requests test * Resolving iframe/pixel priority when iframeEnabled/pixelEnabled * Improving userSync filtering condition * Prioritize iframe user syncing --------- Co-authored-by: Danijel Predarski <[email protected]> Co-authored-by: dani-nova <[email protected]> Co-authored-by: Slavisa Petkovic <[email protected]> Co-authored-by: Slavisa Petkovic <[email protected]>

Added a test case for native ads

…(#10849) * initial commit * adapted buildRequests function * refinement pfilter and bcat * refinement * adapted tests for isBidRequestValid,buildRequests * adaptations for test * finished building stvBidAdapter.js * finished: ran tests, coverage 99% * update: rename w->srw, h->srh * adapt stvBidAdapter.md * remove dspx from stv adapters * some changes (missing: getUserSyncs, but is the same as in radsBidAdapter) * added checks in getUserSyncs; ran tests * added schain support (94.8% coverage) * correct schain encoding * added serializeUids and adapted serializeSChain --------- Co-authored-by: theo_ <theo_@IDEA3>

* update lc version * fix typo

* Update skipRate handling in priceFloors.js and add unit tests for the changes. * Update wording on tests and remove unecessary spread.

…de tracker for pubmatic 3.Skipped adding pubmatic bid in logger (#10897)

…erSyncs (#10800)

* Removed linting issues * Fixed merge issues. * Bugfix on storageTool. * Adnuntius Bid Adapter: bugfix handling ad response Handle no voidAuIds better. --------- Co-authored-by: Mikael Lundin <[email protected]> Co-authored-by: Mikael Lundin <[email protected]>

…sampling and improve transparency (#10792) * track billing events and modify sampling pattern * review updates

* feat: pangle multi format * feat: multi format

…912) Those checks are no longer useful so we decided to remove them.

…TB-conversion-library

* add Rise adapter * fixes * change param isOrg to org * Rise adapter * change email for rise * fix circle failed * bump * bump * bump * remove space * Upgrade Rise adapter to 5.0 * added isWrapper param * addes is_wrapper parameter to documentation * added is_wrapper to test * removed isWrapper * Rise Bid Adapter: support Coppa param (#24) * MinuteMedia Bid Adapter: support Coppa param (#25) * Revert "MinuteMedia Bid Adapter: support Coppa param (#25)" (#26) This reverts commit 66c4e7b46121afc5331c8bca6e2fc972fc55f090. * bump * update coppa fetch * setting coppa param update * update Coppa tests * update test naming * Rise Bid Adapter: support plcmt and sua (#27) * update minuteMediaBidAdapter - support missing params (#29) * support gpp for minutemedia adapter * removed spaces * removed extra character --------- Co-authored-by: Noam Tzuberi <[email protected]> Co-authored-by: noamtzu <[email protected]> Co-authored-by: Noam Tzuberi <[email protected]> Co-authored-by: Laslo Chechur <[email protected]> Co-authored-by: OronW <[email protected]> Co-authored-by: lasloche <[email protected]> Co-authored-by: YakirLavi <[email protected]> Co-authored-by: YakirLavi <[email protected]> Co-authored-by: Inna Yaretsky <>

Co-authored-by: “Nick <“nick.llerandi”@kargo.com>

mamatic and others added 30 commits November 27, 2023 09:00

identityLinkIdSystem: include gpp in 3p envelope retrieval (#10745)

656680b

* identityLinkIdSystem - liveramp - include gpp in 3p envelope retrieval * identityLinkIdSystem - liveramp - adjust test description

Taboola Bid Adapter: cast tmax type (#10766)

b1e601a

* tmax * tmax * tmax * tmax

adxcg Bid Adapter : fix convertTypes util import (#10763)

f215abd

* adxcgBidAdapter: fix convertTypes util import * adxcgBidAdapter: remove previous import

Onetag Bid Adapter: FLEDGE/PAAPI support (#10755)

498b6b5

Co-authored-by: federico <[email protected]>

Grid bid adapter: call data deletion request c2s endpoint (#10732)

6a0c7f0

Co-authored-by: v.raybaud <[email protected]>

SparteoBidAdapter: Send bidder params in OpenRTB request (#10760)

4e323bf

Adagio Bid Adapter: add GPID support (#10775)

b154fc1

Core: add option to turn off Sec-Browsing-Topics header (#10776)

73fb8d3

Missena Bid Adapter: add capping support (#10746)

d70097e

* Missena Bid Adapter: add capping support * Switch to safeJSONParse * Update tests to cover capping

GC-157 Boune request when the PID is missing (#10774)

7dd09d8

Generic analytics: fix spurious warning messages (#10739)

a8fedc6

Prebid 8.26.0 release

5202784

Increment version to 8.27.0-pre

ec42b13

identityLinkIdSystem - liveramp - bug fix setting tcf consent string …

16133c5

…in 3p call (#10778)

Add GVLID to vox adapter (#10784)

faec1bb

LiveIntent UserId module: Add support for sovrn id (#10783)

fe2f35e

* Add EID translation for sovrn * Review comments: fix test

prebidjs-release and others added 4 commits December 28, 2023 20:52

Increment version to 8.30.0-pre

46784ab

refactor using ORTB conversion library

facfcce

Yandex Bid Adapter: add support for topicsFpdModule (#10875)

cc49968

* Yandex Bid Adapter: add support for topicsFpdModule * add test

Yandex Bid Adapter: add jsdoc (#10884)

ff27673

ahmadlob commented Jan 2, 2024

View reviewed changes

kmdevops and others added 23 commits January 2, 2024 12:35

Mygaru Id System: Initial release (#10848)

04a7659

Update viantOrtbBidAdapter_spec.js (#10888)

ed05478

Added a test case for native ads

Missena Bid Adapter: allow per page capping (#10863)

185e97f

LiveIntent ID Module: Update live-connect version (#10894)

49904a0

* update lc version * fix typo

Do not require API for video requests (#10895)

136f0bb

Price Floors: Failure to Account for 'data.skipRate' (#10872)

7a09d81

* Update skipRate handling in priceFloors.js and add unit tests for the changes. * Update wording on tests and remove unecessary spread.

Prebid 8.30.0 release

aa21bfd

Increment version to 8.31.0-pre

714acd3

1.Added safechecks for s2s metadapter case 2.Skipped firing client si…

866be6e

…de tracker for pubmatic 3.Skipped adding pubmatic bid in logger (#10897)

Discovery Bid Adapter: update buyerId, add ssppid & pmguid, add getUs…

60e7e96

…erSyncs (#10800)

ID5 User Id module - pass gpp string and sid in getId request (#10899)

cdbe68c

GreenbidsAnalyticsAdapter and GreenbidsRtdProvider: Rework greenbids …

3e6b546

…sampling and improve transparency (#10792) * track billing events and modify sampling pattern * review updates

Core: add eslint-plugin-jsdoc (#10893)

d390f6c

refactor using ORTB conversion library

3a3482d

Merge branch 'master' into refactor-using-ORTB-conversion-library

a028fc4

Pangle Bid Adapter : add multi format support (#10909)

759c52d

* feat: pangle multi format * feat: multi format

Criteo bid adapter: remove checks on video context and placement (#10…

6689373

…912) Those checks are no longer useful so we decided to remove them.

Merge remote-tracking branch 'upstream/master' into refactor-using-OR…

006f5c3

…TB-conversion-library

refactor using ORTB conversion library

84c8570

refactor using ORTB conversion library

6971737

ahmadlob pushed a commit that referenced this pull request May 19, 2024

refactors code for userIds; updates userId tests (#29) (#11298)

db0b1bd

Co-authored-by: “Nick <“nick.llerandi”@kargo.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor using ORTB conversion library #29

refactor using ORTB conversion library #29

ahmadlob commented Jan 2, 2024

ahmadlob Jan 2, 2024

barRubi Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

ahmadlob Jan 2, 2024

refactor using ORTB conversion library #29

Are you sure you want to change the base?

refactor using ORTB conversion library #29

Conversation

ahmadlob commented Jan 2, 2024

Type of change

Description of change

Other information

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment