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

imds: Rename to advertising

Open pratik-chavan-advertising-dot-com opened this issue 9 months ago • 5 comments

Change the adapter code for imds to advertising reflecting the ownership change from iMedia Digital Services to Advertising.com

Other related PRs: https://github.com/prebid/Prebid.js/pull/12878 https://github.com/prebid/prebid.github.io/pull/5943

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 01cf667797258083f12d9c77bf4887e859d44aca

advertising

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:35:	Builder			100.0%
github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:47:	MakeRequests		100.0%
github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:60:	makeRequest		88.9%
github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:134:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:138:	getExtImpObj		85.7%
github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:157:	MakeBids		100.0%
github.com/prebid/prebid-server/v3/adapters/advertising/advertising.go:195:	getMediaTypeForImp	100.0%
total:										(statements)		94.3%

github-actions[bot] avatar Mar 12 '25 20:03 github-actions[bot]

Hi @pratik-chavan-advertising-dot-com. This is considered a breaking change since you are removing support for the bidder codes imds and syncormedia. As a result, this would need to be part of a major release, such as v4.0, which will be released later this year, perhaps in a few months. If you don't want to wait for the next major release, you should introduce bidder code advertising while continuing to offer support for bidder codes imds and syncormedia for backwards compatibility. One way this could be achieved is to restore imds.yaml and syncormedia.yaml and declare them as hard aliases of advertising via the aliasOf directive.

bsardo avatar Mar 20 '25 17:03 bsardo

Hi @pratik-chavan-advertising-dot-com. This is considered a breaking change since you are removing support for the bidder codes imds and syncormedia. As a result, this would need to be part of a major release, such as v4.0, which will be released later this year, perhaps in a few months. If you don't want to wait for the next major release, you should introduce bidder code advertising while continuing to offer support for bidder codes imds and syncormedia for backwards compatibility. One way this could be achieved is to restore imds.yaml and syncormedia.yaml and declare them as hard aliases of advertising via the aliasOf directive.

@bsardo Thank you so much for the feedback! We would love to get it released sooner for sure. I will make the necessary updates to support backwards compatibility and update the PR soon.

@pratik-chavan-advertising-dot-com , yes. As @bsardo and @bretg mentioned we should wait for major release of prebid-server to avoid impact on existing publishers. So, in short-term

  1. you can introduce new bidder alias.
  2. Keep the same maintainer email till major release

ShriprasadM avatar Apr 08 '25 06:04 ShriprasadM

Maintainer email can change, your contact info in the Prebid documentation can be updated. You can mention the new brand in your Prebid documentation. All cool.

The issue is that name changes are confusing for many - if an old name simply disappears with no reference, people will ask Prebid "what happened to so-and-so". So please plan a transition period where people can access both old and new names. Thank you.

bretg avatar Apr 08 '25 13:04 bretg

@pratik-chavan-advertising-dot-com, I just wanted to check in to see if you had any further questions on the guidance above. Do you have a clear path forward?

bsardo avatar Apr 28 '25 02:04 bsardo

@pratik-chavan-advertising-dot-com : any updates here?

ShriprasadM avatar May 19 '25 16:05 ShriprasadM

@ShriprasadM @bsardo My apologies for late response on this! I'll review the suggestions and update the PR soon. Thank you so much for the feedback.

@pratik-chavan-advertising-dot-com just checking in to see what your plan is going forward given the above comments.

bsardo avatar Jul 24 '25 15:07 bsardo

@bsardo Thank you so much for all the feedback and apologies for the long delay on this. I’ll open a separate PR to implement aliasing while retaining support for the existing bidder codes for backwards compatibility. We can close/decline this PR to keep things clean.

Sounds good @pratik-chavan-advertising-dot-com

bsardo avatar Aug 18 '25 15:08 bsardo