-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
33across id module: does not work in Europe; still sends network request #10495
Comments
related prebid/prebid.github.io#4866 |
@patmmccann Developer from 33Across here. We are looking into the issue regarding US Nat and are working on a fix. |
@patmmccann we released a fix yesterday to our servers and seen a big reduction in the number of US Nat requests dropped. Can you please try again? Also do you mind sharing the details of you tests that resulted in requests being dropped? So we can try them at our end too and make sure we haven't missed anything? Thanks. |
Awesome! We'll send you a test page shortly |
cafedelites.com/?commit=e949a58 |
Thanks! The test page now appears to be making calls and getting proper responses from our endpoint. So I am assuming US Nat is no longer an issue. With respect to GPDR we plan to implement your suggestion #10714 (comment) in that same PR. Our developer working on it will get back to you over there. |
@curlyblueeagle tyvm! I will mark that pr as closes this issue. Thanks for the engagement! |
No problem, thanks for bringing this to our attention. |
@patmmccann We have made the GDPR changes based on what was suggested. Changes have been incorporated to PR 10714 |
Just a reminder that changes are part of PR 10714, please take a look when you have a chance. |
Excellent thanks for your attention on this! |
Type of issue
bug
Description
33across id module has a gvlid and the module passes gdpr consent strings
Should we just delete the gvlid from the module?
Expected results
gdpr enforcement suppresses 33across, or the module suppresses the endpoint call when gdpr applies
Actual results
33across responds with a we do not support this country error
The text was updated successfully, but these errors were encountered: