Skip to content

RTBHouse: Fix unintended site object creation for app requests#4531

Open
pjaworski-rtbh wants to merge 1 commit into
prebid:masterfrom
lotani-rtbh:rtbhouse/site-app-publisherid-bugfix
Open

RTBHouse: Fix unintended site object creation for app requests#4531
pjaworski-rtbh wants to merge 1 commit into
prebid:masterfrom
lotani-rtbh:rtbhouse/site-app-publisherid-bugfix

Conversation

@pjaworski-rtbh

Copy link
Copy Markdown
Contributor

🔧 Type of changes

  • bid adapter update
  • bugfix

✨ What's the context?

The RTBHouse bidder was unconditionally constructing a site object on every outgoing request while attaching the publisher ID. For app-only requests (no site in the incoming Prebid Server request), this resulted in a synthetic site object being added to the outbound bid request, even though only an app object was present.

📋 Description

Fixes publisher ID propagation so it no longer fabricates a site object:

  1. modifySite now only modifies an existing site and returns null untouched — no synthetic site is created when the request has none.
  2. New modifyApp propagates the publisher ID to app.publisher.ext.prebid.publisherId for app traffic, mirroring the existing site.publisher behaviour.
  3. Extracted a shared modifyPublisher helper used by both site and app.

🧪 Testing results

Tests run: 26, Failures: 0, Errors: 0, Skipped: 0
All RTBHouse bidder tests pass, including new coverage:

  • makeHttpRequestsShouldNotCreateSiteWhenBidRequestHasAppOnly — verifies no site is added for app-only requests
  • makeHttpRequestsShouldAddPublisherToAppWhenBidRequestHasAppOnly — verifies publisher ID is propagated to app.publisher
  • Existing site-path and backward-compatibility tests retained

Other information

Please reach us at inventory_support@rtbhouse.com with piotr.jaworski@rtbhouse.com and cc leandro.otani@rtbhouse.com.

@pjaworski-rtbh pjaworski-rtbh marked this pull request as ready for review June 8, 2026 09:53
@pjaworski-rtbh pjaworski-rtbh force-pushed the rtbhouse/site-app-publisherid-bugfix branch from 824fecd to fef41e6 Compare June 8, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant