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

Set Alias doesn't work unless adapter exists #8831

Closed
ghost opened this issue Aug 15, 2022 · 2 comments
Closed

Set Alias doesn't work unless adapter exists #8831

ghost opened this issue Aug 15, 2022 · 2 comments
Labels

Comments

@ghost
Copy link

ghost commented Aug 15, 2022

a## Type of issue
bug?

Description

Seems when we attempt to use aliasBidder function for our bidders that are server side only and do not have an adapter the aliases aren't recognized and we're forced to instead set aliases in the s2sConfig. Seems like this was a change made in the anti-alias feature PR #6150 within modules/prebidServerBidAdapter/index.js around line 558. Was the idea to now only use the s2sConfig when aliasing s2s?

image

Steps to reproduce

Set an alias without the adapter in a Prebid build for a s2s bidder using the following function
pbjs.aliasBidder(adapterName, aliasName);

Expected results

Would expect the alias to be registered

Actual results

Aliases are not getting registered

Other information

@dgirardi
Copy link
Collaborator

Technically reverting this behavior is also a breaking change (sending aliases server side if no client side adapter exists): because if people are currently setting up aliases for other purposes, they could suddenly get picked up by server.

The documentation, as far as I can tell, does say that server-side aliases must be set up through s2sConfig: https://docs.prebid.org/dev-docs/modules/prebidServer.html#server-side-aliases

@ghost
Copy link
Author

ghost commented Aug 17, 2022

Prior to that PR getting added we were always able to set aliases either way. The documentation doesn't seem to say you have to set an alias one way or another. I'm not sure I understand the breaking change you're talking about. Why would someone call for a bidder without an adapter and also not want them server side? Also in order to set a bidder server-side you also have to add them to the list of bidders in the s2sConfig, aliasing them will not automatically send them server-side.

Either way if this is the expected behavior then I think we can clean up some code within adapterManager.aliasBidAdapter where we attempt to set aliases when adapters aren't available.

@ghost ghost mentioned this issue Oct 11, 2022
9 tasks
Repository owner moved this from Needs Req to Done in Prebid.js Tactical Issues table Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

3 participants