Endless Loop in ixBidAdapter with gdprEnforcement module and no consent for purpose 1

[muuki88]

Type of issue

Bug

Description

When upgrading to Prebid 6.1.0 index exchange runs into an endless loop when the gdprEnforcement module is enabled. I wasn't able to check if consent for purpose 1 is given or not, but I assume that it's not.

utils.js:272 Prebid ix ERROR: Error executing handler: events.js RangeError: Maximum call stack size exceeded
    at assign (<anonymous>)
    at _getConfig (config.js:317)
    at Object.getConfig (config.js:351)
    at debugTurnedOn (utils.js:299)
    at Module.logError (utils.js:271)
    at Module.<anonymous> (events.js:70)
    at Array.forEach (<anonymous>)
    at Module._each (utils.js:399)
    at _dispatch (events.js:65)
    at Object.module.exports._public.emit (events.js:99)

Steps to reproduce

I'm not 100% sure about those

1. Upgrade to prebid 6.1.0
2. Add gdprEnforcement module
3. Don't consent to purpose 1

Test page

None.

Expected results

ixAdapter doesn't crash the page with an infinite loop.

Actual results

Chrome crashes

Platform details

• Prebid 6.1.0
• gdprEnforcement module enabled

Other information

• IX Bid Adapter: capture errors in LS and send errors in request #7630 - I assume this commit introduced the bug / @umakajan
• Endless loop in ixBidAdapter #7445

[umakajan]

Thanks @muuki88 for flagging this issue, will take it back to the development team and address the issue.

[muuki88]

Do you still need a test page?

[gglas]

@umakajan Hey guys -- how's progress coming along on this?

[lksharma]

Hey @gglas,
I am looking into this issue. @muuki88 would you be able to provide a test page for my reference?

[muuki88]

Okay. I'm on it. Will be tomorrow though 😄

[muuki88]

Here it is @lksharma

https://frag-muki.de/ix-test/?pbjs_debug=true

Go to privacy settings and remove consent for purpose one. Take a look at the console.

[muuki88]

@lksharma anything I can help you with?

[lksharma]

Hey @muuki88,

I was able to identify the issue and will be pushing my changes shortly. Once I have the branch ready would you be able to help verify against your test page?

[muuki88]

Sure thing

[lksharma]

@muuki88 I pushed my changes here. I've verified the fix on my end. Let me know if you have any issues with your tests as well. Thank you.

[muuki88]

@lksharma works 🎉