MobileFuse Adapter: Remove tagid_src and pub_id params

src/main/java/org/prebid/server/bidder/mobilefuse/MobilefuseBidder.java

@@ -45,17 +45,6 @@ public MobilefuseBidder(String endpointUrl, JacksonMapper mapper) {
 
     @Override
     public Result<List<HttpRequest<BidRequest>>> makeHttpRequests(BidRequest request) {
-        final String endpoint = request.getImp().stream()
-                .map(this::parseImpExt)
-                .filter(Objects::nonNull)
-                .findFirst()
-                .map(this::makeUrl)
-                .orElse(null);
-
-        if (endpoint == null) {
-            return Result.withError(BidderError.badInput("Invalid ExtImpMobilefuse value"));
-        }
-
         final List<Imp> modifiedImps = request.getImp().stream()
                 .map(this::modifyImp)
                 .filter(Objects::nonNull)
@@ -66,7 +55,7 @@ public Result<List<HttpRequest<BidRequest>>> makeHttpRequests(BidRequest request
         }
 
         final BidRequest modifiedRequest = request.toBuilder().imp(modifiedImps).build();
-        return Result.withValue(BidderUtil.defaultRequest(modifiedRequest, endpoint, mapper));
+        return Result.withValue(BidderUtil.defaultRequest(modifiedRequest, endpointUrl, mapper));
     }
 
     private Imp modifyImp(Imp imp) {
@@ -75,6 +64,10 @@ private Imp modifyImp(Imp imp) {
         }
 
         final ExtImpMobilefuse impExt = parseImpExt(imp);
+        if (impExt == null) {
+            return null;
+        }
+
         final ObjectNode skadn = parseSkadn(imp.getExt());
         return imp.toBuilder()
                 .tagid(Objects.toString(impExt.getPlacementId(), "0"))
@@ -98,11 +91,6 @@ private ObjectNode parseSkadn(ObjectNode impExt) {
         }
     }
 
-    private String makeUrl(ExtImpMobilefuse extImp) {
-        final String baseUrl = endpointUrl + Objects.toString(extImp.getPublisherId(), "0");
-        return "ext".equals(extImp.getTagidSrc()) ? baseUrl + "&tagid_src=ext" : baseUrl;
-    }
-
     @Override
     public final Result<List<BidderBid>> makeBids(BidderCall<BidRequest> httpCall, BidRequest bidRequest) {
         try {

src/main/java/org/prebid/server/proto/openrtb/ext/request/mobilefuse/ExtImpMobilefuse.java

@@ -13,10 +13,4 @@ public class ExtImpMobilefuse {
 
     @JsonProperty("placement_id")
     Integer placementId;
-
-    @JsonProperty("pub_id")
-    Integer publisherId;
-
-    @JsonProperty("tagid_src")
-    String tagidSrc;
 }

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

@@ -1,6 +1,6 @@
 adapters:
   mobilefuse:
-    endpoint: http://mfx.mobilefuse.com/openrtb?pub_id=
+    endpoint: http://mfx.mobilefuse.com/openrtb
     ortb-version: "2.6"
     endpoint-compression: gzip
     # This bidder does not operate globally. Please consider setting "disabled: true" outside of the following regions:

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

@@ -7,18 +7,9 @@
     "placement_id": {
       "type": "integer",
       "description": "An ID which identifies this specific inventory placement"
-    },
-    "pub_id": {
-      "type": "integer",
-      "description": "An ID which identifies the publisher selling the inventory."
-    },
-    "tagid_src": {
-      "type": "string",
-      "description": "ext if passing publisher's ids, empty if passing MobileFuse IDs in placement_id field. Defaults to empty"
     }
   },
   "required": [
-    "placement_id",
-    "pub_id"
+    "placement_id"
   ]
 }

src/test/java/org/prebid/server/bidder/mobilefuse/MobilefuseBidderTest.java

@@ -36,15 +36,15 @@
 
 public class MobilefuseBidderTest extends VertxTest {
 
-    private static final String ENDPOINT_URL = "https://test.endpoint.com/openrtb?pub_id=";
+    private static final String ENDPOINT_URL = "https://test.endpoint.com/openrtb";
 
     private final MobilefuseBidder target = new MobilefuseBidder(ENDPOINT_URL, jacksonMapper);
 
     private static Imp givenImp(Function<Imp.ImpBuilder, Imp.ImpBuilder> impCustomizer) {
         return impCustomizer.apply(Imp.builder()
                         .id("imp_id")
                         .ext(mapper.valueToTree(ExtPrebid.of(null,
-                                ExtImpMobilefuse.of(1, 2, "tagidSrc")))))
+                                ExtImpMobilefuse.of(1)))))
                 .build();
     }
 
@@ -92,7 +92,7 @@ public void makeHttpRequestsShouldReturnErrorIfNoValidExtFound() {
 
         // then
         assertThat(result.getErrors())
-                .containsExactly(BidderError.badInput("Invalid ExtImpMobilefuse value"));
+                .containsExactly(BidderError.badInput("No valid imps"));
         assertThat(result.getValue()).isEmpty();
     }
 
@@ -102,44 +102,6 @@ public void creationShouldFailOnInvalidEndpointUrl() {
                 .isThrownBy(() -> new MobilefuseBidder("invalid_url", jacksonMapper));
     }
 
-    @Test
-    public void makeHttpRequestsShouldCreateCorrectURLWhenTagidSrcEqualsExt() {
-        // given
-        final BidRequest bidRequest = givenBidRequest(
-                impBuilder -> impBuilder
-                        .banner(Banner.builder().build())
-                        .ext(mapper.valueToTree(ExtPrebid.of(null,
-                                ExtImpMobilefuse.of(1, 2, "ext")))));
-
-        // when
-        final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);
-
-        // then
-        assertThat(result.getErrors()).isEmpty();
-        assertThat(result.getValue())
-                .extracting(HttpRequest::getUri)
-                .containsExactly("https://test.endpoint.com/openrtb?pub_id=2&tagid_src=ext");
-    }
-
-    @Test
-    public void makeHttpRequestsShouldSetPubIdToZeroIfPublisherIdNotPresentInRequest() {
-        // given
-        final BidRequest bidRequest = givenBidRequest(
-                impBuilder -> impBuilder
-                        .banner(Banner.builder().build())
-                        .ext(mapper.valueToTree(ExtPrebid.of(null,
-                                ExtImpMobilefuse.of(1, null, null)))));
-
-        // when
-        final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);
-
-        // then
-        assertThat(result.getErrors()).isEmpty();
-        assertThat(result.getValue())
-                .extracting(HttpRequest::getUri)
-                .containsExactly("https://test.endpoint.com/openrtb?pub_id=0");
-    }
-
     @Test
     public void makeHttpRequestsShouldReturnErrorIfNoValidImpsFound() {
         // given
@@ -193,7 +155,7 @@ public void makeHttpRequestsShouldModifyImpTagId() {
                         .banner(Banner.builder().build())
                         .tagid("some tag id")
                         .ext(mapper.valueToTree(ExtPrebid.of(null,
-                                ExtImpMobilefuse.of(1, 2, "ext")))));
+                                ExtImpMobilefuse.of(1)))));
 
         // when
         final Result<List<HttpRequest<BidRequest>>> result = target.makeHttpRequests(bidRequest);
@@ -212,7 +174,7 @@ public void makeHttpRequestsShouldModifyImpWithAddingSkadnWhenSkadnIsPresent() {
         // given
         final ObjectNode skadn = mapper.createObjectNode().put("something", "something");
         final ObjectNode impExt = mapper.createObjectNode();
-        impExt.set("bidder", mapper.valueToTree(ExtImpMobilefuse.of(1, 2, "ext")));
+        impExt.set("bidder", mapper.valueToTree(ExtImpMobilefuse.of(1)));
         impExt.set("skadn", skadn);
         final BidRequest bidRequest = givenBidRequest(
                 impBuilder -> impBuilder.banner(Banner.builder().build()).ext(impExt));

src/test/java/org/prebid/server/it/MobilefuseTest.java

@@ -18,7 +18,7 @@ public class MobilefuseTest extends IntegrationTest {
     @Test
     public void openrtb2AuctionShouldRespondWithBidsFromMobilefuse() throws IOException, JSONException {
         // given
-        WIRE_MOCK_RULE.stubFor(post(urlPathEqualTo("/mobilefuse-exchange/1111&tagid_src=ext"))
+        WIRE_MOCK_RULE.stubFor(post(urlPathEqualTo("/mobilefuse-exchange/"))
                 .withRequestBody(equalToJson(jsonFrom("openrtb2/mobilefuse/test-mobilefuse-bid-request.json")))
                 .willReturn(aResponse().withBody(jsonFrom("openrtb2/mobilefuse/test-mobilefuse-bid-response.json"))));
 

src/test/resources/org/prebid/server/it/openrtb2/mobilefuse/test-auction-mobilefuse-request.json

@@ -10,9 +10,7 @@
       "tagid": "tag_id",
       "ext": {
         "mobilefuse": {
-          "placement_id": 999999,
-          "pub_id": 1111,
-          "tagid_src": "ext"
+          "placement_id": 999999
         }
       }
     }