9001 / 9001/copyparty

The NixOS module has multiple issues

Offen
#1,634 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug help wanted
Vorherrschende Sprache
Python
Sterne
46.6k
Forks
1.9k
Ø Merge
10 T. 16 Std.
Gemergte PRs (30 T.)
5

Beschreibung

### Describe the bug
As seen [here](https://github.com/9001/copyparty/blob/hovudstraum/contrib/nixos/modules/copyparty.nix#L121C7-L125C9), some settings are set by default, and notably a pretty important one, `hist`, that sets the cache dir location.

When the user change the settings, the values get overriden, thus breaks thumbnail generation because it then tries to write thumbnails into a .hist folder at the root of the volume, which copyparty needs write access to do so (not the case by default).

Additionally, the module sets some values, such as externalCacheDir or externalStateDir. Not only the user can't override these values, this also means that a user changing the location of `services.copyparty.settings.hist` wouldn't automatically add it to the `BindPaths` of the systemd service, so copyparty wouldn't have access to the folder.

There are probably more issues, this is what I quickly found after trying to find out why my thumbnails wouldn't generate.

### To Reproduce
Set `services.copyparty.settings` to anything, thumbnail generation breaks.
Set `services.copyparty.settings.hist` to anything other than `/var/cache/copyparty`, the systemd service doesn't bind mount that path.

### Expected behavior
The default values of these options should remain (but should be overridable), even if the user sets `services.copyparty.settings`, and the user should be able to change the cache and state directories, which are then reflected into the `config` block of the module.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.