nicktarallo commented on PR #773: URL: https://github.com/apache/commons-vfs/pull/773#issuecomment-4977165356
> Hello @nicktarallo Thank you for the PR. Please review the comments from copilot. You might need to add a test in check the main suggestion. The test suggestion seems like a test bug. Hi @garydgregory, thanks for the review. I addressed both comments: - Authority detection now only triggers on a leading // (network-path reference) or on :// following the scheme. Added regression tests for the file:/a//[...] example from the review, a :// embedded in the path after the authority, and a URI mixing an IPv6 zone ID with brackets in the path. - Wrapped the Files.walk stream in try-with-resources. Also verified all 4 new test methods fail against the pre-fix code. -- 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]
