Suggestion: ignore dotfiles in `conf.d`

[huyz]

Is your feature request related to a problem? Please describe.

I have special files in all my bind-mount directories, including the conf.d/ directory where docker-volume-backup reads backup configuration files. Since these files are unrecognized, DVB won't start up.

Describe the solution you'd like

Ignore all dotfiles, which are traditionally considered hidden files, in conf.d:

docker-volume-backup/cmd/backup/config_provider.go

Line 100 in 87bf4fb

for _, item := range items {

Describe alternatives you've considered

Looks like the code skips directories, so I may create a special subdirectory to hide my dotfiles.

[m90]

The problem with excluding dotfiles by default is that some people might already rely on the current behavior. So, if this were to be added, it would need to be put behind a configuration option. I have to admit though that I'm not entirely sure that's worth it as it seems to be very edge-casey.

How hard would it be for you to put the dotfiles into a directory as you describe instead?

[huyz]

It would seem odd to rely on the current behavior when their filenames start with ., wouldn't it? That seems even more edge-casey than the common convention that programs doing file globbing should by default ignore dotfiles unless they really mean to read them.

It wouldn't be hard for me. This is more for future folks who could run into the gotcha.

Feel free to close.

[m90]

Incidentally, I have one particular deployment myself where config files are named .foo.env and so on, so maybe I'm biased as it'd break my very own setup :)

I'll leave this open, so others can chime in if this is a problem for them as well.