From e5cae733b174d87fc31d9e8dfec1e7436252ab4f Mon Sep 17 00:00:00 2001
From: Willy Kloucek <wkloucek@owncloud.com>
Date: Mon, 14 Mar 2022 14:21:37 +0100
Subject: [PATCH 1/2] fix audit config docs

---
 audit/cmd/notifications/main.go            |  4 +-
 audit/pkg/config/defaultconfig.go          | 18 --------
 audit/pkg/config/defaults/defaultconfig.go | 49 ++++++++++++++++++++++
 audit/pkg/config/parser/parse.go           | 15 ++-----
 4 files changed, 55 insertions(+), 31 deletions(-)
 delete mode 100644 audit/pkg/config/defaultconfig.go
 create mode 100644 audit/pkg/config/defaults/defaultconfig.go

diff --git a/audit/cmd/notifications/main.go b/audit/cmd/notifications/main.go
index 64d0cdc081c..05322e138e4 100644
--- a/audit/cmd/notifications/main.go
+++ b/audit/cmd/notifications/main.go
@@ -4,11 +4,11 @@ import (
 	"os"
 
 	"github.com/owncloud/ocis/audit/pkg/command"
-	"github.com/owncloud/ocis/audit/pkg/config"
+	"github.com/owncloud/ocis/idm/pkg/config/defaults"
 )
 
 func main() {
-	if err := command.Execute(config.DefaultConfig()); err != nil {
+	if err := command.Execute(defaults.DefaultConfig()); err != nil {
 		os.Exit(1)
 	}
 }
diff --git a/audit/pkg/config/defaultconfig.go b/audit/pkg/config/defaultconfig.go
deleted file mode 100644
index 7988ab3fd15..00000000000
--- a/audit/pkg/config/defaultconfig.go
+++ /dev/null
@@ -1,18 +0,0 @@
-package config
-
-func DefaultConfig() *Config {
-	return &Config{
-		Service: Service{
-			Name: "audit",
-		},
-		Events: Events{
-			Endpoint:      "127.0.0.1:9233",
-			Cluster:       "test-cluster",
-			ConsumerGroup: "audit",
-		},
-		Auditlog: Auditlog{
-			LogToConsole: true,
-			Format:       "json",
-		},
-	}
-}
diff --git a/audit/pkg/config/defaults/defaultconfig.go b/audit/pkg/config/defaults/defaultconfig.go
new file mode 100644
index 00000000000..eb3dd9de8aa
--- /dev/null
+++ b/audit/pkg/config/defaults/defaultconfig.go
@@ -0,0 +1,49 @@
+package defaults
+
+import (
+	"github.com/owncloud/ocis/audit/pkg/config"
+)
+
+func FullDefaultConfig() *config.Config {
+	cfg := DefaultConfig()
+
+	EnsureDefaults(cfg)
+	Sanitize(cfg)
+
+	return cfg
+}
+
+func DefaultConfig() *config.Config {
+	return &config.Config{
+		Service: config.Service{
+			Name: "audit",
+		},
+		Events: config.Events{
+			Endpoint:      "127.0.0.1:9233",
+			Cluster:       "test-cluster",
+			ConsumerGroup: "audit",
+		},
+		Auditlog: config.Auditlog{
+			LogToConsole: true,
+			Format:       "json",
+		},
+	}
+}
+
+func EnsureDefaults(cfg *config.Config) {
+	// provide with defaults for shared logging, since we need a valid destination address for BindEnv.
+	if cfg.Log == nil && cfg.Commons != nil && cfg.Commons.Log != nil {
+		cfg.Log = &config.Log{
+			Level:  cfg.Commons.Log.Level,
+			Pretty: cfg.Commons.Log.Pretty,
+			Color:  cfg.Commons.Log.Color,
+			File:   cfg.Commons.Log.File,
+		}
+	} else if cfg.Log == nil {
+		cfg.Log = &config.Log{}
+	}
+}
+
+func Sanitize(cfg *config.Config) {
+	// sanitize config
+}
diff --git a/audit/pkg/config/parser/parse.go b/audit/pkg/config/parser/parse.go
index 6bcaf0ad757..51be3a53ca3 100644
--- a/audit/pkg/config/parser/parse.go
+++ b/audit/pkg/config/parser/parse.go
@@ -4,6 +4,7 @@ import (
 	"errors"
 
 	"github.com/owncloud/ocis/audit/pkg/config"
+	"github.com/owncloud/ocis/audit/pkg/config/defaults"
 	ociscfg "github.com/owncloud/ocis/ocis-pkg/config"
 
 	"github.com/owncloud/ocis/ocis-pkg/config/envdecode"
@@ -16,17 +17,7 @@ func ParseConfig(cfg *config.Config) error {
 		return err
 	}
 
-	// provide with defaults for shared logging, since we need a valid destination address for BindEnv.
-	if cfg.Log == nil && cfg.Commons != nil && cfg.Commons.Log != nil {
-		cfg.Log = &config.Log{
-			Level:  cfg.Commons.Log.Level,
-			Pretty: cfg.Commons.Log.Pretty,
-			Color:  cfg.Commons.Log.Color,
-			File:   cfg.Commons.Log.File,
-		}
-	} else if cfg.Log == nil {
-		cfg.Log = &config.Log{}
-	}
+	defaults.EnsureDefaults(cfg)
 
 	// load all env variables relevant to the config in the current context.
 	if err := envdecode.Decode(cfg); err != nil {
@@ -36,5 +27,7 @@ func ParseConfig(cfg *config.Config) error {
 		}
 	}
 
+	defaults.Sanitize(cfg)
+
 	return nil
 }

From 899510b158dad52bab24f7a957af968d9a1a5ae7 Mon Sep 17 00:00:00 2001
From: Willy Kloucek <wkloucek@owncloud.com>
Date: Mon, 14 Mar 2022 15:00:46 +0100
Subject: [PATCH 2/2] fix config package import

---
 audit/cmd/notifications/main.go  | 2 +-
 ocis-pkg/config/defaultconfig.go | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/audit/cmd/notifications/main.go b/audit/cmd/notifications/main.go
index 05322e138e4..68f15f670df 100644
--- a/audit/cmd/notifications/main.go
+++ b/audit/cmd/notifications/main.go
@@ -4,7 +4,7 @@ import (
 	"os"
 
 	"github.com/owncloud/ocis/audit/pkg/command"
-	"github.com/owncloud/ocis/idm/pkg/config/defaults"
+	"github.com/owncloud/ocis/audit/pkg/config/defaults"
 )
 
 func main() {
diff --git a/ocis-pkg/config/defaultconfig.go b/ocis-pkg/config/defaultconfig.go
index 9767f2d9335..07ce28407e6 100644
--- a/ocis-pkg/config/defaultconfig.go
+++ b/ocis-pkg/config/defaultconfig.go
@@ -2,7 +2,7 @@ package config
 
 import (
 	accounts "github.com/owncloud/ocis/accounts/pkg/config/defaults"
-	audit "github.com/owncloud/ocis/audit/pkg/config"
+	audit "github.com/owncloud/ocis/audit/pkg/config/defaults"
 	glauth "github.com/owncloud/ocis/glauth/pkg/config/defaults"
 	graphExplorer "github.com/owncloud/ocis/graph-explorer/pkg/config/defaults"
 	graph "github.com/owncloud/ocis/graph/pkg/config/defaults"