OMS: add video support

src/main/java/org/prebid/server/bidder/oms/OmsBidder.java

@@ -1,6 +1,9 @@
 package org.prebid.server.bidder.oms;
 
+import com.fasterxml.jackson.core.type.TypeReference;
 import com.iab.openrtb.request.BidRequest;
+import com.iab.openrtb.request.Imp;
+import com.iab.openrtb.response.Bid;
 import com.iab.openrtb.response.BidResponse;
 import com.iab.openrtb.response.SeatBid;
 import org.apache.commons.collections4.CollectionUtils;
@@ -10,9 +13,13 @@
 import org.prebid.server.bidder.model.BidderError;
 import org.prebid.server.bidder.model.HttpRequest;
 import org.prebid.server.bidder.model.Result;
+import org.prebid.server.exception.PreBidException;
 import org.prebid.server.json.DecodeException;
 import org.prebid.server.json.JacksonMapper;
+import org.prebid.server.proto.openrtb.ext.ExtPrebid;
+import org.prebid.server.proto.openrtb.ext.request.omx.ExtImpOms;
 import org.prebid.server.proto.openrtb.ext.response.BidType;
+import org.prebid.server.proto.openrtb.ext.response.ExtBidPrebidVideo;
 import org.prebid.server.util.BidderUtil;
 import org.prebid.server.util.HttpUtil;
 
@@ -23,6 +30,8 @@
 
 public class OmsBidder implements Bidder<BidRequest> {
 
+    private static final TypeReference<ExtPrebid<?, ExtImpOms>> EXT_TYPE_REFERENCE = new TypeReference<>() {
+    };
     private final String endpointUrl;
     private final JacksonMapper mapper;
 
@@ -32,16 +41,39 @@ public OmsBidder(String endpointUrl, JacksonMapper mapper) {
     }
 
     @Override
-    public final Result<List<HttpRequest<BidRequest>>> makeHttpRequests(BidRequest bidRequest) {
-        return Result.withValue(BidderUtil.defaultRequest(bidRequest, endpointUrl, mapper));
+    public Result<List<HttpRequest<BidRequest>>> makeHttpRequests(BidRequest request) {
+        if (!request.getImp().isEmpty()) {
+            try {
+                final ExtImpOms impExt = parseImpExt(request.getImp().getFirst());
+                final String publisherId = impExt.getPid() == null
+                        && impExt.getPublisherId() != null
+                        && impExt.getPublisherId() > 0
+                        ? String.valueOf(impExt.getPublisherId())
+                        : impExt.getPid();
+                final String url = "%s?publisherId=%s".formatted(endpointUrl, publisherId);
+                return Result.withValue(BidderUtil.defaultRequest(request, url, mapper));
+            } catch (PreBidException e) {
+                return Result.withError(BidderError.badInput(e.getMessage()));
+            }
+        }
+
+        return Result.withValue(BidderUtil.defaultRequest(request, endpointUrl, mapper));
+    }
+
+    private ExtImpOms parseImpExt(Imp imp) throws PreBidException {
+        try {
+            return mapper.mapper().convertValue(imp.getExt(), EXT_TYPE_REFERENCE).getBidder();
+        } catch (IllegalArgumentException e) {
+            throw new PreBidException("Invalid ext. Imp.Id: " + imp.getId());
+        }
     }
 
     @Override
     public final Result<List<BidderBid>> makeBids(BidderCall<BidRequest> httpCall, BidRequest bidRequest) {
         try {
             final BidResponse bidResponse = mapper.decodeValue(httpCall.getResponse().getBody(), BidResponse.class);
             return Result.withValues(extractBids(bidResponse));
-        } catch (DecodeException e) {
+        } catch (DecodeException | PreBidException e) {
             return Result.withError(BidderError.badServerResponse(e.getMessage()));
         }
     }
@@ -59,7 +91,37 @@ private static List<BidderBid> bidsFromResponse(BidResponse bidResponse) {
                 .map(SeatBid::getBid)
                 .filter(Objects::nonNull)
                 .flatMap(Collection::stream)
-                .map(bid -> BidderBid.of(bid, BidType.banner, bidResponse.getCur()))
+                .map(bid -> BidderBid.builder()
+                        .bid(bid)
+                        .type(getBidType(bid))
+                        .bidCurrency(bidResponse.getCur())
+                        .videoInfo(videoInfo(bid))
+                        .build())
                 .toList();
     }
+
+    private static BidType getBidType(Bid bid) {
+        final Integer markupType = bid.getMtype();
+        return switch (markupType) {
+            case 1 -> BidType.banner;
+            case 2 -> BidType.video;
+            case null, default -> BidType.banner;
+        };
+    }
+
+    private static ExtBidPrebidVideo videoInfo(Bid bid) {
+        if (!Integer.valueOf(2).equals(bid.getMtype())) {
+            return null;
+        }
+        final List<String> cat = bid.getCat();
+        final Integer duration = bid.getDur();
+
+        final boolean catNotEmpty = CollectionUtils.isNotEmpty(cat);
+        final boolean durationValid = duration != null && duration > 0;
+        return catNotEmpty || durationValid
+                ? ExtBidPrebidVideo.of(
+                durationValid ? duration : null,
+                catNotEmpty ? cat.getFirst() : null)
+                : null;
+    }
 }

src/main/java/org/prebid/server/proto/openrtb/ext/request/omx/ExtImpOms.java

@@ -0,0 +1,12 @@
+package org.prebid.server.proto.openrtb.ext.request.omx;
+
+import com.fasterxml.jackson.annotation.JsonProperty;
+import lombok.Value;
+
+@Value(staticConstructor = "of")
+public class ExtImpOms {
+
+    String pid;
+    @JsonProperty("publisherId")
+    Integer publisherId;
+}

src/main/resources/bidder-config/oms.yaml

@@ -5,7 +5,9 @@ adapters:
       maintainer-email: prebid@onlinemediasolutions.com
       app-media-types:
         - banner
+        - video
       site-media-types:
         - banner
+        - video
       supported-vendors:
       vendor-id: 0

src/main/resources/static/bidder-params/oms.json

@@ -6,11 +6,25 @@
   "properties": {
     "pid": {
       "type": "string",
-      "description": "An id used to identify OMS publisher.",
+      "description": "Deprecated: An id used to identify OMS publisher.",
       "minLength": 5
+    },
+    "publisherId": {
+      "type": "integer",
+      "description": "An ID used to identify OMS publisher.",
+      "minimum": 10000
     }
   },
-  "required": [
-    "pid"
+  "oneOf": [
+    {
+      "required": [
+        "pid"
+      ]
+    },
+    {
+      "required": [
+        "publisherId"
+      ]
+    }
   ]
 }