slachiewicz opened a new pull request, #12843: URL: https://github.com/apache/maven/pull/12843
Make sure `AuthenticationContext` instances are properly closed even when exceptions are thrown during property retrieval, using try-with-resources. This is a modern rebase / successor of stale PR #669 targeting the active `master` branch: - The `AuthenticationContext` try-with-resources conversions in `LegacyRepositorySystem` and `MavenRepositorySystem` have been updated for current paths (`compat/maven-compat` and `impl/maven-core`). - Added try-with-resources conversion for `LegacyRepositorySystemSessionExtender`. - The obsolete conversions from #669 (`ProjectBuildingException`, `DefaultMavenPluginManager`) were omitted as those classes were already refactored. Closes #669 - [x] I hereby declare this contribution to be licenced under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0) -- 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]
