LouisLau-art opened a new pull request, #16116: URL: https://github.com/apache/dubbo/pull/16116
## What Fixes an input stream resource leak in `ConfigUtils.loadProperties(...)` when `allowMultiFile == false` and fallback loading path is used. Previously the code called: - `properties.load(ClassUtils.getClassLoader().getResourceAsStream(fileName));` This could leave the stream unclosed. The fallback now uses try-with-resources and null-checks the stream before loading. ## Why This addresses issue #16098 and keeps resource handling consistent with other load paths in `ConfigUtils`. ## Verification - `./mvnw -pl dubbo-common -Dtest=ConfigUtilsTest test -DskipITs` - Result: BUILD SUCCESS (`Tests run: 25, Failures: 0, Errors: 0, Skipped: 1`) Fixes #16098 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
