Copper6 adapter: change utility suite #14991
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ee4645d5a1
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| bids: [ | ||
| { | ||
| bidder: 'copper6ssp', | ||
| params: { |
There was a problem hiding this comment.
What are these params, it seems In order to not break existing integrations you'll need to figure out some sort of translation layer
There was a problem hiding this comment.
We explain this params in Documentation
https://github.com/prebid/prebid.github.io/pull/6597/changes
We have added these type recently
@typedef {import('./copper6sspBidAdapter.d.ts').Copper6SSPBidRequestParams} Copper6SSPBidRequestParams
I thought that this file was requested for additional clarity of the code.
Also I have changed the "ext" object and added comment to have some explanation. Do you think it is clearer now?
There was a problem hiding this comment.
your doc file has the wrong name, you can't force pubs to change params in a minor version so you should figure out how to translate what they do now to what you want
There was a problem hiding this comment.
I see. I will work on it. Thank you
There was a problem hiding this comment.
@patmmccann we have internal discussion and have talked to Copper6 team to understand the impact on the partners that use the adapter. Copper6 team took responsibility of the requested change. The request initially came from them and they will manage all publishers accordingly. Please see the attached conversation.
Can it be possible from this perspective to approve the change we want to make in minor version?

There was a problem hiding this comment.
Hi @patmmccann
Is there any chance that the awareness of the partner from Copper6 (see my previous comment) can make this pr appropriate for a minor version?
If not - do you have any idea when v12 is expected to be published?
There was a problem hiding this comment.
Even if they made it themselves, the issue is it is breaking; why doesn't perion do a server side translation of the requests without the required params? If there's only 18 pubs, surely you can ease their pain by inferring a cid and pid server side from their referer URL and placementId?
There was a problem hiding this comment.
We expect to publish v12 in May or June of 2027
There was a problem hiding this comment.
@patmmccann
We had find the way to accept old style bid requests on our end and changed the adapter accordingly.
We do not want to wait till the v12. :)
Please review.
Barecheck - Code coverage reportTotal: 96.58%Your code coverage diff: 0.01% ▴ Uncovered files and lines
|
…s for copper6ssp bid adapter
# Conflicts: # test/spec/modules/copper6sspBidAdapter_spec.js
|
Tread carefully! This PR adds 5 linter errors (possibly disabled through directives):
|
|
This PR introduces changes that may not work on all browsers. According to Babel, the following polyfills may be needed, and they are not automatically included:
The best way to address this is to provide good test coverage, as normal PR checks run unit tests on older browsers. |
|
This PR introduces changes that may not work on all browsers. According to Babel, the following polyfills may be needed, and they are not automatically included:
The best way to address this is to provide good test coverage, as normal PR checks run unit tests on older browsers. |
Type of change
Bugfix
Feature
New bidder adapter
Updated bidder adapter
Code style update (formatting, local variables)
Refactoring (no functional changes, no api changes)
Build related changes
CI related changes
Does this change affect user-facing APIs or examples documented on http://prebid.org?
Other
Description of change
Other information
We have received the following email that approves start of our maintenance:
Perion Mail - Fwd_ Copper6 Prebid Adapter - now Operated by Vidazoo _ Perion.pdf
Related PRs:
prebid/prebid.github.io#6597
prebid/prebid-server#4812