Skip to content
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

Adyoulike adapter user sync pixel causes popup message in Safari #2919

Closed
mkarlov opened this issue Jul 12, 2023 · 7 comments
Closed

Adyoulike adapter user sync pixel causes popup message in Safari #2919

mkarlov opened this issue Jul 12, 2023 · 7 comments
Labels

Comments

@mkarlov
Copy link

mkarlov commented Jul 12, 2023

When prebid server loads usersync URL https://acdn.adnxs.com/prebid/universal-creative/load-cookie.html?max_sync_count=30
and if it loads usersync pixels from bidder adapter Adyoulike https:/prebid/prebid-server/blob/master/static/bidder-info/adyoulike.yaml the url prompts popup message in Safari browser.

Example:
{
"bidder": "adyoulike",
"no_cookie": true,
"usersync": {
"url": "https://visitor.omnitagjs.com/visitor/isync?uid=19340f4f097d16f41f34fc0274981ca4&name=PrebidServer&gdpr=&gdpr_consent=&us_privacy=&url=https%3A%2F%2Fib.adnxs.com%2Fprebid%2Fsetuid%3Fbidder%3Dadyoulike%26gdpr%3D%26gdpr_consent%3D%26gpp%3D%26gpp_sid%3D%26f%3Db%26uid%3D%5BBUYER_USERID%5D",
"type": "iframe"
}
},

To reproduce, paste the url in Safari. This causes bad user experience and complaints from website visitors.

@bretg
Copy link
Contributor

bretg commented Jul 12, 2023

@mkarlov - I'm not getting an error in safari for this URL. Please paste the error or a screenshot.

Heads up @SimonThevenot and @chrisg93 - may be something AdYouLike should look at.

@mkarlov
Copy link
Author

mkarlov commented Jul 12, 2023

Hi @bretg

Attaching screenshots from my test and client live site.

Screenshot 2023-07-12 at 15 39 39 Screenshot 2023-07-12 at 15 41 50

omni

@bretg bretg added the bug label Jul 12, 2023
@bretg
Copy link
Contributor

bretg commented Jul 12, 2023

Thanks. @SimonThevenot and @chrisg93 - I suspect this might have to do with your isync script not having a content-type tag on the response. (?)

@SimonThevenot
Copy link
Contributor

Hello @bretg @mkarlov. Thanks for raising this issue. Yep it seems we are missing a header, we'll take care of it as soon as possible, I'll keep you in touch

@SimonThevenot
Copy link
Contributor

Hello @bretg @mkarlov
This has been fixed, could you confirm that the behavior is now OK ? Thanks

@bretg
Copy link
Contributor

bretg commented Jul 27, 2023

@mkarlov - the isync script now returns a content-type. Am going to close this as resolved. Please re-open if needed.

@bretg bretg closed this as completed Jul 27, 2023
@mkarlov
Copy link
Author

mkarlov commented Aug 8, 2023

Sorry for the late response, I was away for the last couple of weeks. The issue appears to be fixed now, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

3 participants