From 03b9d50c676ae5008c99d1fdb4606d1a95c9e5b2 Mon Sep 17 00:00:00 2001 From: Earl Warren Date: Sun, 11 Aug 2024 14:16:48 +0200 Subject: [PATCH] fix: enable LOG_COMPRESSION by default Refs: https://codeberg.org/forgejo/forgejo/pulls/4924#issuecomment-2165839 --- custom/conf/app.example.ini | 2 +- modules/setting/actions.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/custom/conf/app.example.ini b/custom/conf/app.example.ini index e414ee8ab0..239ae527a0 100644 --- a/custom/conf/app.example.ini +++ b/custom/conf/app.example.ini @@ -2717,7 +2717,7 @@ LEVEL = Info ;; It's always recommended to use compression when using local disk as log storage if CPU or memory is not a bottleneck. ;; And for object storage services like S3, which is billed for requests, it would cause extra 2 times of get requests for each log view. ;; But it will save storage space and network bandwidth, so it's still recommended to use compression. -;LOG_COMPRESSION = none +;LOG_COMPRESSION = zstd ;; Default artifact retention time in days. Artifacts could have their own retention periods by setting the `retention-days` option in `actions/upload-artifact` step. ;ARTIFACT_RETENTION_DAYS = 90 ;; Timeout to stop the task which have running status, but haven't been updated for a long time diff --git a/modules/setting/actions.go b/modules/setting/actions.go index fdc285428b..8c1b57b649 100644 --- a/modules/setting/actions.go +++ b/modules/setting/actions.go @@ -58,11 +58,11 @@ func (c logCompression) IsValid() bool { } func (c logCompression) IsNone() bool { - return c == "" || strings.ToLower(string(c)) == "none" + return strings.ToLower(string(c)) == "none" } func (c logCompression) IsZstd() bool { - return strings.ToLower(string(c)) == "zstd" + return c == "" || strings.ToLower(string(c)) == "zstd" } func loadActionsFrom(rootCfg ConfigProvider) error {