| 5 Mar 2022 |
 Winter (she/her) | Riiight, completely forgot about that.
I think the best thing to do here is to revisit how the Caddy module operates in this regard -- so removing the blanket "acme" group addition. (Since I'm not sure the best way to do this, would it be appropriate to open an issue to discuss it with the module maintainer?) | 17:36:06 |
 m1cr0man | Yeah that's probably best, and so that it's on record on Github too | 17:44:17 |
| Winter (she/her) | In reply to @m1cr0man:m1cr0man.com
Yeah that's probably best, and so that it's on record on Github too Would it be appropriate to label the issue as a bug? | 19:34:00 |
| Winter (she/her) | (don't wanna open an issue with no label idk) | 19:34:07 |
| Winter (she/her) | i think so | 19:34:22 |
| m1cr0man | hah uh idk what label to use honestly 😅 I think it's more just discussion atm, nothing is wrong per se | 19:34:33 |
| m1cr0man | * hah uh idk what label to use honestly 😅 I think it's more just discussion/suggestion atm, nothing is wrong per se | 19:34:40 |
| Winter (she/her) | true | 19:34:43 |
| Winter (she/her) | yeah ill do no label | 19:34:53 |
| Winter (she/her) | m1cr0man: am i just blind, or is the group option for not defined in certOpts? | 19:51:09 |
| Winter (she/her) | * m1cr0man: am i just blind, or is the group option not defined in certOpts? | 19:51:14 |
| m1cr0man | it's defined in the inheritableModule thing | 19:51:23 |
| Winter (she/her) | oh | 19:51:53 |
| Winter (she/her) | group = mkOption {
type = types.str;
inherit (defaultAndText "group" "acme") default defaultText;
description = "Group running the ACME client.";
};
i feel like this description is inaccurate?
| 19:52:02 |
| Winter (she/her) | oh nevermind | 19:52:24 |
| Winter (she/her) | guess its not | 19:52:27 |
| Winter (she/her) | # Group might change between runs, re-apply it
chown '${user}:${data.group}' certificates/*
hm
| 19:52:57 |
| m1cr0man | yeah that's 100% necessary | 19:53:11 |
| m1cr0man | ran into it myself and covered by the test suite | 19:53:19 |
| Winter (she/her) | so is that if the certificate doest have to be renewed, but the group changed? | 19:53:33 |
