yybmion commented on code in PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#discussion_r2289835030
##########
log4j-core/src/main/java/org/apache/logging/log4j/core/config/ConfigurationFactory.java:
##########
@@ -617,4 +636,19 @@ private String[] parseConfigLocations(final String
configLocations) {
static List<ConfigurationFactory> getFactories() {
return factories;
}
+
+ /**
+ * Returns all configuration file extensions currently supported at
runtime.
+ * This aggregates extensions from all active configuration factories.
+ *
+ * @return list of supported file extensions
+ * @since 2.25.0
+ */
+ public static List<String> getActiveFileExtensions() {
Review Comment:
Thanks for the review @vy !
You're absolutely right about the redundancy. The static method duplicates
the functionality already provided by the instance method.
Therefore, I think it would be better to remove the static
`getActiveFileExtensions()` method and use the instance method approach instead.
--
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: notifications-unsubscr...@logging.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
