dxbjavid commented on PR #745: URL: https://github.com/apache/commons-text/pull/745#issuecomment-4591933495
Added one. The existing `../` tests only passed because the escaped file didn't exist, so the read failed and got rethrown as IllegalArgumentException anyway. The new test in FileStringLookupTest points a relative `..` path at a real file outside the fence: on current main the fence lets it through and the file is read (nothing thrown), with the fix it's rejected. -- 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]
