Conversant bid adapter to use userIdAsEids (prebid#4967)

* added support for pubcommon, digitrust, id5id

* added support for IdentityLink

* changed the source for id5

* added unit test cases

* changed source param for identityLink

* using userIdAsEids

Author: pm-harshad-mane

modules/conversantBidAdapter.js

@@ -314,36 +314,24 @@ function copyOptProperty(src, dst, dstName) {
 function collectEids(bidRequests) {
   const request = bidRequests[0]; // bidRequests have the same userId object
   const eids = [];
-
-  addEid(eids, request, 'userId.tdid', 'adserver.org');
-  addEid(eids, request, 'userId.idl_env', 'liveramp.com');
-  addEid(eids, request, 'userId.criteoId', 'criteo.com');
-  addEid(eids, request, 'userId.id5id', 'id5-sync.com');
-  addEid(eids, request, 'userId.parrableid', 'parrable.com');
-  addEid(eids, request, 'userId.digitrustid.data.id', 'digitru.st');
-  addEid(eids, request, 'userId.lipb.lipbid', 'liveintent.com');
-
-  return eids;
-}
-
-/**
- * Extract and push a single extended id into eids array
- * @param eids Array of extended IDs
- * @param idObj Object containing IDs
- * @param keyPath Nested properties expressed as a path
- * @param source Source for the ID
- */
-function addEid(eids, idObj, keyPath, source) {
-  const id = utils.deepAccess(idObj, keyPath);
-  if (id) {
-    eids.push({
-      source: source,
-      uids: [{
-        id: id,
-        atype: 1
-      }]
+  if (utils.isArray(request.userIdAsEids) && request.userIdAsEids.length > 0) {
+    // later following white-list can be converted to block-list if needed
+    const requiredSourceValues = {
+      'adserver.org': 1,
+      'liveramp.com': 1,
+      'criteo.com': 1,
+      'id5-sync.com': 1,
+      'parrable.com': 1,
+      'digitru.st': 1,
+      'liveintent.com': 1
+    };
+    request.userIdAsEids.forEach(function(eid) {
+      if (requiredSourceValues.hasOwnProperty(eid.source)) {
+        eids.push(eid);
+      }
     });
   }
+  return eids;
 }
 
 /**

test/spec/modules/conversantBidAdapter_spec.js

@@ -1,6 +1,7 @@
 import {expect} from 'chai';
 import {spec} from 'modules/conversantBidAdapter.js';
 import * as utils from 'src/utils.js';
+import { createEidsArray } from 'modules/userId/eids.js';
 
 describe('Conversant adapter tests', function() {
   const siteId = '108060';
@@ -395,6 +396,7 @@ describe('Conversant adapter tests', function() {
     // add pubcid to every entry
     requests.forEach((unit) => {
       Object.assign(unit, {userId: {pubcid: 67890}});
+      Object.assign(unit, {userIdAsEids: createEidsArray(unit.userId)});
     });
     //  construct http post payload
     const payload = spec.buildRequests(requests).data;
@@ -469,11 +471,12 @@ describe('Conversant adapter tests', function() {
       // add pubcid to every entry
       requests.forEach((unit) => {
         Object.assign(unit, {userId: {pubcid: 112233, tdid: 223344, idl_env: 334455}});
+        Object.assign(unit, {userIdAsEids: createEidsArray(unit.userId)});
       });
       //  construct http post payload
       const payload = spec.buildRequests(requests).data;
       expect(payload).to.have.deep.nested.property('user.ext.eids', [
-        {source: 'adserver.org', uids: [{id: 223344, atype: 1}]},
+        {source: 'adserver.org', uids: [{id: 223344, atype: 1, ext: {rtiPartner: 'TDID'}}]},
         {source: 'liveramp.com', uids: [{id: 334455, atype: 1}]}
       ]);
     });