prebid-server icon indicating copy to clipboard operation
prebid-server copied to clipboard

Sovrn Adapter has a type mismatch with the bidfloor parameter

Open premesh opened this issue 1 year ago • 11 comments

If the bidfloor parameter is used as defined in the prebid.js docs but sent to PBS you get the following error message image

Looks like the PBS Java Sovrn Adapter is looking for a number https://github.com/prebid/prebid-server-java/blob/70f5b9ad65796559d3ff884fe917dd41bfaa786b/src/main/resources/static/bidder-params/sovrn.json#L15

While the PBJS Sovrn Adapter expects a string https://docs.prebid.org/dev-docs/bidders/sovrn.html image

premesh avatar Feb 20 '24 02:02 premesh

@ikagotso - can you take a look at this? The best solution, IMO, is to remove the bidfloor parameter entirely. Your adapter should be using the Prebid Server-wide floors feature.

bretg avatar Feb 20 '24 19:02 bretg

Note, this is also a string in PBS-Go https://github.com/prebid/prebid-server/blob/master/static/bidder-params/sovrn.json

bretg avatar Feb 23 '24 15:02 bretg

Friendly ping @ikagotso, have you had time to look at this issue?

SyntaxNode avatar Mar 08 '24 15:03 SyntaxNode

@SyntaxNode We have started discussions around this.

ikagotso avatar Mar 11 '24 10:03 ikagotso

We have opened a PR with a fix.

ikagotso avatar Mar 14 '24 11:03 ikagotso

PR to fix this is already merged.

ikagotso avatar Mar 18 '24 11:03 ikagotso

Which PR was is @ikagotso ? I don't see it either PBS-Go or PBS-Java repo

bretg avatar Mar 18 '24 12:03 bretg

@bretg https://github.com/prebid/Prebid.js/pull/11208

ikagotso avatar Mar 18 '24 12:03 ikagotso

Thanks @ikagotso -- that may be helpful in the long run, but publishers don't upgrade Prebid.js often so the mismatch is going to persist for years in the wild. You're going to want to update your server-side adapter accept either string or number since Prebid Servers are upgraded much more frequently by host companies.

This can be done in in the JSON file https://github.com/prebid/prebid-server/blob/master/static/bidder-params/sovrn.json and by writing a unit test to make sure the adapter itself handles the scenario.

bretg avatar Mar 18 '24 12:03 bretg

@ikagotso - this is still an issue in both PBS-Go and PBS-Java. Please fix your adapter:

  1. Remove the bidfloor parameter entirely.

OR

  1. Update the parameter schema to accept bidfloor as either string or number
  2. Update adapter to convert the bidfloor to whatever your endpoint expects

bretg avatar Aug 23 '24 14:08 bretg

@bretg Sorry for delay. We will start working on the fix in a short while.

ikagotso avatar Sep 26 '24 15:09 ikagotso