DaanHoogland commented on PR #7980: URL: https://github.com/apache/cloudstack/pull/7980#issuecomment-1728964922
@GutoVeronezi I am not against making it standard. I am against not proxying it. In a codebase as big as this one we need to be aware of the maintenance costs of 3rd library updates. We should proxy all important libraries, even if only passing through. This is also why I think the log4j change is half done. It is a best practice for using 3rd party libraries that we only half implement in our project (i.e. hypervisor plugins) and should be more vigilent about. Strings are not the most complex 3rd party library but still... anyway, not -1íng just warning for a breaking with best practices here. -- 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]
