search

logstash-plugins / logstash-output-file

Apache License 2.0
23 stars 53 forks source link

Add symlink traversal #89

Open jrcichra opened 4 years ago

jrcichra commented 4 years ago

This PR allows pre-existing, strategically placed symlinks, to no longer fail on mkdir_p calls.

I've replaced mkdir_p with a loop per directory that only creates the directory if the directory doesn't exist and is not a symlink.

This solves: https://github.com/logstash-plugins/logstash-output-file/issues/87

CLA is signed (Corporation + Individual under corporation [NetApp] applies here).

jrcichra commented 3 years ago

@roaksoax Could someone please review this? Thanks.

jrcichra commented 3 years ago

@jsvd @roaksoax Could this please be reviewed? We'd really like to have this feature upstream. Thanks.

jonathonpuff commented 3 years ago

@jsvd @roaksoax looking for your help here.

jonathonpuff commented 3 years ago

@jsvd @roaksoax checking back in to see if we get this merged.