Skip to content

Commit 3a3a036

Browse files
authored
Merge pull request #49419 from gastaldi/npe
Enforce null checks for `DiscoveredServiceBuilder`
2 parents b206715 + 16afa02 commit 3a3a036

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

core/deployment/src/main/java/io/quarkus/deployment/builditem/DevServicesResultBuildItem.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import java.util.Collections;
66
import java.util.HashMap;
77
import java.util.Map;
8+
import java.util.Objects;
89
import java.util.function.Consumer;
910
import java.util.function.Function;
1011
import java.util.function.Supplier;
@@ -190,7 +191,7 @@ public static class DiscoveredServiceBuilder {
190191
private String description;
191192

192193
public DiscoveredServiceBuilder name(String name) {
193-
this.name = name;
194+
this.name = Objects.requireNonNull(name, "name cannot be null");
194195
return this;
195196
}
196197

@@ -215,6 +216,9 @@ public DiscoveredServiceBuilder description(String description) {
215216
}
216217

217218
public DevServicesResultBuildItem build() {
219+
if (name == null) {
220+
throw new IllegalStateException("name cannot be null");
221+
}
218222
return new DevServicesResultBuildItem(name, description, containerId, config);
219223
}
220224
}

0 commit comments

Comments
 (0)