Skip to content

Commit 1962e8e

Browse files
committed
only allow comma as delimiter for lists in .databrickscfg
1 parent 1a694e9 commit 1962e8e

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

databricks-sdk-java/src/main/java/com/databricks/sdk/core/ConfigAttributeAccessor.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,14 +69,15 @@ public void setValueOnConfig(DatabricksConfig cfg, String value) throws IllegalA
6969
field.set(cfg, ProxyConfig.ProxyAuthType.valueOf(value));
7070
} else if (List.class.isAssignableFrom(field.getType())) {
7171
// Handle List<String> fields (e.g., scopes)
72-
// Parse comma and/or whitespace separated values from environment variable or config file.
72+
// Parse comma-separated values from environment variable or config file.
7373
if (field.getGenericType() instanceof ParameterizedType) {
7474
ParameterizedType paramType = (ParameterizedType) field.getGenericType();
7575
if (paramType.getActualTypeArguments().length > 0
7676
&& paramType.getActualTypeArguments()[0] == String.class) {
77-
// Split by commas and/or whitespace and filter out empty strings
77+
// Split by comma, trim each value, and filter out empty strings
7878
List<String> list =
79-
Arrays.stream(value.trim().split("[,\\s]+"))
79+
Arrays.stream(value.trim().split(","))
80+
.map(String::trim)
8081
.filter(s -> !s.isEmpty())
8182
.collect(Collectors.toList());
8283
field.set(cfg, list);

0 commit comments

Comments
 (0)