Skip to content

Commit 1928473

Browse files
Bugfix: Null handling to not throw exceptions (prebid#3035)
1 parent fe98db2 commit 1928473

3 files changed

Lines changed: 8 additions & 1 deletion

File tree

src/main/java/org/prebid/server/auction/gpp/SetuidGppService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package org.prebid.server.auction.gpp;
22

33
import io.vertx.core.Future;
4+
import org.apache.commons.lang3.StringUtils;
45
import org.prebid.server.auction.gpp.model.GppContext;
56
import org.prebid.server.auction.gpp.model.GppContextCreator;
67
import org.prebid.server.auction.gpp.model.GppContextWrapper;
@@ -44,7 +45,7 @@ private static GppContextWrapper contextFrom(PrivacyContext privacyContext) {
4445

4546
private static Integer toInt(String string) {
4647
try {
47-
return string != null ? Integer.parseInt(string) : null;
48+
return StringUtils.isNotBlank(string) ? Integer.parseInt(string) : null;
4849
} catch (NumberFormatException e) {
4950
return null;
5051
}

src/main/java/org/prebid/server/bidder/pubmatic/PubmaticBidder.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,9 @@ private BigDecimal resolveBidFloor(String kadfloor, BigDecimal existingFloor) {
214214
}
215215

216216
private static BigDecimal parseKadFloor(String kadFloorString) {
217+
if (StringUtils.isBlank(kadFloorString)) {
218+
return null;
219+
}
217220
try {
218221
return new BigDecimal(StringUtils.trimToEmpty(kadFloorString));
219222
} catch (NumberFormatException e) {

src/main/java/org/prebid/server/privacy/ccpa/Ccpa.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,9 @@ public boolean isNotEmpty() {
2626
}
2727

2828
public boolean isEnforced() {
29+
if (usPrivacy == null) {
30+
return false;
31+
}
2932
try {
3033
validateUsPrivacy(usPrivacy);
3134
} catch (PreBidException e) {

0 commit comments

Comments
 (0)