vy commented on code in PR #3921:
URL: https://github.com/apache/logging-log4j2/pull/3921#discussion_r2419281369
##########
log4j-core/src/main/java/org/apache/logging/log4j/core/config/ConfigurationFactory.java:
##########
@@ -333,6 +334,52 @@ public Configuration getConfiguration(
return getConfiguration(loggerContext, name, configLocation);
}
+ /**
+ * Creates a Configuration from multiple configuration URIs.
+ * If multiple URIs are successfully loaded, they will be combined into a
CompositeConfiguration.
+ *
+ * @param loggerContext the logger context (may be null)
+ * @param name the configuration name (may be null)
+ * @param uris the list of configuration URIs (must not be null or empty)
+ * @return a Configuration created from the provided URIs
+ * @throws NullPointerException if uris is null
+ * @throws IllegalArgumentException if uris is empty
+ * @throws ConfigurationException if no valid configuration could be
created
+ * from any of the provided URIs
+ * @since 2.26.0
+ */
+ public Configuration getConfiguration(final LoggerContext loggerContext,
final String name, final List<URI> uris) {
+
+ Objects.requireNonNull(uris, "uris parameter cannot be null");
+
+ if (uris.isEmpty()) {
+ throw new IllegalArgumentException("URI list cannot be empty");
+ }
Review Comment:
I'm fine with delegating to `getConfiguration(loggerContext, name, (URI)
null)`, but don't return `null` please.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
