EndzeitBegins commented on PR #8503:
URL: https://github.com/apache/nifi/pull/8503#issuecomment-2020788649

   I haven't encountered issues regarding symbolic links in the (arguably 
limited) manual testing I've done. 
   
   However, I do not quite understand what you're addressing. 
   The PR does not change the way `getListing` addresses the SFTP server. 
`stat` is only used when inquiring for a single resource using 
`getRemoteFileInfo`. 
   
   The only change in `getListing` is how the results are mapped. However, even 
that's not really the case. Where before `newFileInfo` would receive a 
`RemoteResourceInfo` and access `.getAttributes()`, `.getName()`, and 
`.isDirectory()` it now receives the first two as arguments directly. The third 
one was just a utility function that accessed the same attributes under the 
hood which is basically inlined. 
   I'm unsure how this would affect `getListing` to be honest. 🤔 


-- 
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]

Reply via email to