ramitg254 commented on PR #471: URL: https://github.com/apache/tez/pull/471#issuecomment-4142460110
> > > 1 more thing to check: I think a path like `../newfile` should also not work, because it would let users to write to places where they are not supposed to > > > > > > I think it will be fine as it will be resolved to valid path by getCanonicalFile() before checks are run on it > > yeah, it might be the case, but it should be validated with a unit test case also done added -- 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]
