[Bidder Adapter] Add AdPlayx adapter by anuraj700 · Pull Request #4582 · prebid/prebid-server-java

anuraj700 · 2026-07-31T15:39:54Z

Type of Change

New bidder adapter
Bug fix
Feature / Enhancement
Documentation update

Description

This PR adds the AdPlayx bidder adapter implementation (adplayx) to Prebid Server Java, supporting ORTB 2.6 requests for Banner and Video inventory.

Key Changes Made:

Added AdplayxBidder.java to handle request construction with query parameters (apptoken, placementid) and bid response parsing.
Added ExtImpAdplayx.java model for extracting adapter parameters.
Added bidder configuration (adplayx.yaml, AdplayxConfiguration.java).
Added parameter validation schema (static/bidder-params/adplayx.json).
Added unit test suite AdplayxBidderTest.java targeting >80% code coverage using JUnit 5 and AssertJ.

Checklist

Code passes checkstyle:check without line length or final modifier violations.
Unit tests pass locally (mvn test -Dtest=AdplayxBidderTest).
Parameters match JSON schema defined in static/bidder-params/adplayx.json.
Added configuration in bidder-config/adplayx.yaml.

Testing Instructions

Run unit tests locally:

mvn test -Dtest=AdplayxBidderTest

### 🔧 Type of changes
- [ ] new bid adapter
- [ ] bid adapter update
- [ ] new feature
- [ ] new analytics adapter
- [ ] new module
- [ ] module update
- [ ] bugfix
- [ ] documentation
- [ ] configuration
- [ ] dependency update
- [ ] tech debt (test coverage, refactorings, etc.)

### ✨ What's the context?
What's the context for the changes?

### 🧠 Rationale behind the change
Why did you choose to make these changes? Were there any trade-offs you had to consider?

### 🔎 New Bid Adapter Checklist
- [ ] verify email contact works
- [ ] NO fully dynamic hostnames
- [ ] geographic host parameters are NOT required
- [ ] direct use of HTTP is prohibited - *implement an existing Bidder interface that will do all the job*
- [ ] if the ORTB is just forwarded to the endpoint, use the generic adapter - *define the new adapter as the alias of the generic adapter*
- [ ] cover an adapter configuration with an integration test

### 🧪 Test plan
How do you know the changes are safe to ship to production?

### 🏎 Quality check
- [ ] Are your changes following [our code style guidelines](https://github.com/prebid/prebid-server-java/blob/master/docs/developers/code-style.md)?
- [ ] Are there any breaking changes in your code?
- [ ] Does your test coverage exceed 90%?
- [ ] Are there any erroneous console logs, debuggers or leftover code in your changes?

## Type of Change - [x] New bidder adapter - [ ] Bug fix - [ ] Feature / Enhancement - [ ] Documentation update ## Description This PR adds the **AdPlayx** bidder adapter implementation (`adplayx`) to Prebid Server Java, supporting ORTB 2.6 requests for Banner and Video inventory. ### Key Changes Made: - Added `AdplayxBidder.java` to handle request construction with query parameters (`apptoken`, `placementid`) and bid response parsing. - Added `ExtImpAdplayx.java` model for extracting adapter parameters. - Added bidder configuration (`adplayx.yaml`, `AdplayxConfiguration.java`). - Added parameter validation schema (`static/bidder-params/adplayx.json`). - Added unit test suite `AdplayxBidderTest.java` targeting >80% code coverage using JUnit 5 and AssertJ. ## Checklist - [x] Code passes `checkstyle:check` without line length or final modifier violations. - [x] Unit tests pass locally (`mvn test -Dtest=AdplayxBidderTest`). - [x] Parameters match JSON schema defined in `static/bidder-params/adplayx.json`. - [x] Added configuration in `bidder-config/adplayx.yaml`. ## Testing Instructions Run unit tests locally: ```bash mvn test -Dtest=AdplayxBidderTest

CTMBNara

Update branch with latest master. Use Uri.java instead of macros (it will encode all the params, so make sure to not double encode them).

- Remove {{apptoken}} macro template from bidder config - Append apptoken and placementid dynamically using URIBuilder - Update unit tests to verify parameter building and single-encoding behavior

…/prebid-server-java into adding-adapter-adplayx

anuraj700 · 2026-08-15T05:51:33Z

Update branch with latest master. Use Uri.java instead of macros (it will encode all the params, so make sure to not double encode them).

Resolved the review comments.

Replace org.apache.http.client.utils.URIBuilder with org.prebid.server.util.Uri in AdplayxBidder to eliminate the deprecated Apache HTTP client dependency. - Updated buildEndpointUrl() to use Uri.of() and addQueryParam() - Updated AdplayxBidderTest assertions to align with Vert.x UriTemplate encoding style

CTMBNara

Add integration test. Use AxisTest as an example

CTMBNara · 2026-08-25T16:22:18Z

+public class AdplayxBidder implements Bidder<BidRequest> {
+
+    private static final TypeReference<ExtPrebid<?, ExtImpAdplayx>> ADPLAYX_EXT_TYPE_REFERENCE =
+            new TypeReference<>() { };


Format new TypeReference<>() { } according to project style

CTMBNara · 2026-08-25T16:23:02Z

+    }
+
+    private String buildEndpointUrl(ExtImpAdplayx extImp) {
+        return Uri.of(endpointUrl)


Move Uri.of(...) creation to adapter constructor

CTMBNara · 2026-08-25T16:23:39Z

+
+                final String uri = buildEndpointUrl(extImp);
+
+                // Clone bid request for this impression


remove comment

CTMBNara · 2026-08-25T16:24:12Z

+                httpRequests.add(HttpRequest.<BidRequest>builder()
+                        .method(HttpMethod.POST)
+                        .uri(uri)
+                        .headers(HttpUtil.headers())
+                        .body(mapper.encodeToBytes(outgoingRequest))
+                        .payload(outgoingRequest)
+                        .build());


Use BidderUtil.defaultRequest

CTMBNara · 2026-08-25T16:25:27Z

+        if (StringUtils.isBlank(responseBody)) {
+            return Result.empty();
+        }


No need for this check. Inline responseBody

CTMBNara · 2026-08-25T16:29:07Z

+        try {
+            final BidResponse bidResponse = mapper.decodeValue(responseBody, BidResponse.class);
+            if (bidResponse == null || bidResponse.getSeatbid() == null) {
+                return Result.empty();
+            }
+
+            final List<BidderError> errors = new ArrayList<>();
+            final List<BidderBid> bidderBids = new ArrayList<>();
+
+            for (final SeatBid seatBid : bidResponse.getSeatbid()) {
+                for (final Bid bid : seatBid.getBid()) {
+                    try {
+                        final BidType bidType = getBidType(bid.getImpid(), bidRequest.getImp());
+                        bidderBids.add(BidderBid.of(bid, bidType, bidResponse.getCur()));
+                    } catch (final PreBidException e) {
+                        errors.add(BidderError.badServerResponse(e.getMessage()));
+                    }
+                }
+            }
+
+            return Result.of(bidderBids, errors);
+        } catch (final Exception e) {
+            return Result.withError(BidderError.badServerResponse("Failed to decode response: " + e.getMessage()));
+        }
+    }
+
+    private BidType getBidType(String impId, List<Imp> imps) {
+        for (Imp imp : imps) {
+            if (imp.getId().equals(impId)) {
+                if (imp.getBanner() != null) {
+                    return BidType.banner;
+                }
+                if (imp.getVideo() != null) {
+                    return BidType.video;
+                }
+                if (imp.getAudio() != null) {
+                    return BidType.audio;
+                }
+                if (imp.getXNative() != null) {
+                    return BidType.xNative;
+                }
+            }
+        }
+        throw new PreBidException("Failed to find impression with id: " + impId);
+    }


Refactor this code to be similar to other adapters. See AxisBidder as an example

CTMBNara · 2026-08-25T16:30:28Z

@@ -0,0 +1,16 @@
+adapters:
+  adplayx:
+    enabled: true


Remove enabled: true

CTMBNara · 2026-08-25T16:32:10Z

+      app-media-types:
+        - banner
+        - video
+      site-media-types:
+        - banner
+        - video


I see a discrepancy between the code (supports all media types) and the supported media types (banner + video). If it's intended - then ok

CTMBNara requested changes Aug 7, 2026

View reviewed changes

anuraj700 added 3 commits August 15, 2026 10:56

Merge branch 'prebid:master' into adding-adapter-adplayx

4b6de5d

adplayx: construct request URI using URIBuilder instead of macros

87a6ff3

- Remove {{apptoken}} macro template from bidder config - Append apptoken and placementid dynamically using URIBuilder - Update unit tests to verify parameter building and single-encoding behavior

Merge branch 'adding-adapter-adplayx' of https://github.com/anuraj700…

b4bf265

…/prebid-server-java into adding-adapter-adplayx

anuraj700 requested a review from CTMBNara August 15, 2026 05:50

CTMBNara requested changes Aug 19, 2026

View reviewed changes

Comment thread src/main/java/org/prebid/server/bidder/adplayx/AdplayxBidder.java Outdated

anuraj700 requested a review from CTMBNara August 20, 2026 14:11

CTMBNara requested changes Aug 25, 2026

View reviewed changes

Navigation Menu

Uh oh!

[Bidder Adapter] Add AdPlayx adapter - #4582

[Bidder Adapter] Add AdPlayx adapter#4582
anuraj700 wants to merge 5 commits into
prebid:masterfrom
anuraj700:adding-adapter-adplayx

anuraj700 commented Jul 31, 2026

Uh oh!

CTMBNara left a comment

Uh oh!

anuraj700 commented Aug 15, 2026

Uh oh!

Uh oh!

CTMBNara left a comment

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

CTMBNara Aug 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants


		final String uri = buildEndpointUrl(extImp);

		// Clone bid request for this impression

Uh oh!

Conversation

anuraj700 commented Jul 31, 2026

Type of Change

Description

Key Changes Made:

Checklist

Testing Instructions

Uh oh!

CTMBNara left a comment

Choose a reason for hiding this comment

Uh oh!

anuraj700 commented Aug 15, 2026

Uh oh!

Uh oh!

CTMBNara left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants