fix: skip unreadable files in mounted plugins dir

tedtramonte commented 2 years ago

I tried my hardest to keep it a simple one-liner but find really does not like dealing with files without x permissions, so I gave in and went with a simple for loop.

Tested locally and my issue in #206 was resolved: a mounted directory with a lost+found dir with perms of 700 no longer prevented the container from starting, and a plugin next to that lost+found dir was successfully placed into plugins-merged and owned by graylog:graylog.

Closes #206

Notes for Reviewers

[ ] The commit history must be preserved - please use the rebase-merge or standard merge option instead of squash-merge
[ ] Sync up with the author before merging

CLAassistant commented 2 years ago

All committers have signed the CLA.

tedtramonte commented 2 years ago

@bernd Don't mean to bother, is there anything else you need from me on this?

mpfz0r commented 1 year ago

@tedtramonte

Thank you. I've merged you fixes into a new PR: https://github.com/Graylog2/graylog-docker/pull/216

Graylog2 / graylog-docker

fix: skip unreadable files in mounted plugins dir #208

Notes for Reviewers