Why digitrust callback triggered twice ?

[ckybonist]

Type of issue

bug and or question

Description

The callback was triggered twice by userId module.

pbjs.setConfig({
  userSync: {
    userIds: [{
      name: "pubCommonId",
      storage: {
        type: "cookie",
        name: "_pubcid",       // create a cookie with this name
        expires: 365           // expires in 1 years
      }
    }, {
      name: "digitrust",
      params: {
        init: {
          member: 'example_member_id',
          site: 'example_site_id'
        },
        callback: function (digiTrustResult) {
          if (digiTrustResult.success) {
            console.log('Success in Digitrust init', digiTrustResult.identity.id);
          } else {
            console.error('Digitrust init failed');
          }
        }
      },
      storage: {
        type: "html5",
        name: "pbjsdigitrust",
        expires: 60
      }
    }]
  }});

Steps to reproduce

Just use the example on userId document

Test page

• https://jsfiddle.net/kychu/hc2eb960/10/
• Open console and you'll see two messages:

Success in Digitrust init
xxxxxxxxx

NOTE: xxxxxx means the id value

Expected results

Only executed / triggered the callback once.

Actual results

Callback has been executed twice.

Platform details

• Prebid.js: 2.44.0
• JSFiddle Prebid.js: https://acdn.adnxs.com/prebid/not-for-prod/1/prebid.js
• Browser: Google Chrome 79.0.3945.117

Other information

[bretg]

@goosemanjack - can you take a look?

[goosemanjack]

Will do. Will update this ticket when I find something.

…

On Tue, Jan 14, 2020 at 7:07 AM bretg ***@***.***> wrote: @goosemanjack <https://github.com/goosemanjack> - can you take a look? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#4728?email_source=notifications&email_token=AAFC44WFS7YHNLCVTWHW65LQ5XIMZA5CNFSM4KGNSW6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI46BJY#issuecomment-574218407>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFC44QE5RWVHXLJDEKYU6LQ5XIMZANCNFSM4KGNSW6A> .

[goosemanjack]

PR #4745
Thank you for reporting this. This PR fixes the issue.