Merge pull request #147498 from helsinki-systems/feat/disable-acme-debugging
nixos/acme: Disable bash tracing
This commit is contained in:
commit
e8862a91ff
1 changed files with 6 additions and 1 deletions
|
@ -325,7 +325,8 @@ let
|
||||||
|
|
||||||
# Working directory will be /tmp
|
# Working directory will be /tmp
|
||||||
script = ''
|
script = ''
|
||||||
set -euxo pipefail
|
${optionalString data.enableDebugLogs "set -x"}
|
||||||
|
set -euo pipefail
|
||||||
|
|
||||||
# This reimplements the expiration date check, but without querying
|
# This reimplements the expiration date check, but without querying
|
||||||
# the acme server first. By doing this offline, we avoid errors
|
# the acme server first. By doing this offline, we avoid errors
|
||||||
|
@ -438,6 +439,8 @@ let
|
||||||
default = "_mkMergedOptionModule";
|
default = "_mkMergedOptionModule";
|
||||||
};
|
};
|
||||||
|
|
||||||
|
enableDebugLogs = mkEnableOption "debug logging for this certificate" // { default = cfg.enableDebugLogs; };
|
||||||
|
|
||||||
webroot = mkOption {
|
webroot = mkOption {
|
||||||
type = types.nullOr types.str;
|
type = types.nullOr types.str;
|
||||||
default = null;
|
default = null;
|
||||||
|
@ -616,6 +619,8 @@ in {
|
||||||
options = {
|
options = {
|
||||||
security.acme = {
|
security.acme = {
|
||||||
|
|
||||||
|
enableDebugLogs = mkEnableOption "debug logging for all certificates by default" // { default = true; };
|
||||||
|
|
||||||
validMinDays = mkOption {
|
validMinDays = mkOption {
|
||||||
type = types.int;
|
type = types.int;
|
||||||
default = 30;
|
default = 30;
|
||||||
|
|
Loading…
Reference in a new issue