Skip to content

Commit 7323099

Browse files
committed
Code cleanup
1 parent 3119f39 commit 7323099

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

src/main/java/org/prebid/server/spring/config/ServiceConfiguration.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1034,8 +1034,8 @@ RequestValidator requestValidator(
10341034
JacksonMapper mapper,
10351035
@Value("${logging.sampling-rate:0.01}") double logSamplingRate,
10361036
@Value("${auction.strict-app-site-dooh:false}") boolean enabledStrictAppSiteDoohValidation,
1037-
@Value("${settings.fail-on-disabled-bidders}") boolean failOnDisabledBidders,
1038-
@Value("${settings.fail-on-unknown-bidders}") boolean failOnUnknownBidders) {
1037+
@Value("${settings.fail-on-disabled-bidders:true}") boolean failOnDisabledBidders,
1038+
@Value("${settings.fail-on-unknown-bidders:true}") boolean failOnUnknownBidders) {
10391039

10401040
return new RequestValidator(
10411041
bidderCatalog,

src/main/java/org/prebid/server/validation/RequestValidator.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ public ValidationResult validate(Account account,
134134
validateTargeting(targeting);
135135
}
136136
aliases = ObjectUtils.defaultIfNull(extRequestPrebid.getAliases(), Collections.emptyMap());
137-
validateAliases(aliases, warnings, metrics, account);
137+
validateAliases(aliases, warnings, account);
138138
validateAliasesGvlIds(extRequestPrebid, aliases);
139139
validateBidAdjustmentFactors(extRequestPrebid.getBidadjustmentfactors(), aliases);
140140
validateExtBidPrebidData(extRequestPrebid.getData(), aliases, isDebugEnabled, warnings);
@@ -514,16 +514,16 @@ private static void validateGranularityRangeIncrement(ExtGranularityRange range)
514514
* is equals to itself.
515515
*/
516516
private void validateAliases(Map<String, String> aliases, List<String> warnings,
517-
Metrics metrics, Account account) throws ValidationException {
517+
Account account) throws ValidationException {
518518

519519
for (final Map.Entry<String, String> aliasToBidder : aliases.entrySet()) {
520520
final String alias = aliasToBidder.getKey();
521521
final String coreBidder = aliasToBidder.getValue();
522522
if (!bidderCatalog.isValidName(coreBidder)) {
523523
metrics.updateUnknownBidderMetric(account);
524524

525-
final String message = String.format("request.ext.prebid.aliases.%s refers to unknown bidder: %s",
526-
alias, coreBidder);
525+
final String message = "request.ext.prebid.aliases.%s refers to unknown bidder: %s".formatted(alias,
526+
coreBidder);
527527
if (failOnUnknownBidders) {
528528
throw new ValidationException(message);
529529
} else {
@@ -532,8 +532,8 @@ private void validateAliases(Map<String, String> aliases, List<String> warnings,
532532
} else if (!bidderCatalog.isActive(coreBidder)) {
533533
metrics.updateDisabledBidderMetric(account);
534534

535-
final String message = String.format("request.ext.prebid.aliases.%s refers to disabled bidder: %s",
536-
alias, coreBidder);
535+
final String message = "request.ext.prebid.aliases.%s refers to disabled bidder: %s".formatted(alias,
536+
coreBidder);
537537
if (failOnDisabledBidders) {
538538
throw new ValidationException(message);
539539
} else {

0 commit comments

Comments
 (0)