new config framework in ocs

ocis/pkg/command/ocs.go

@@ -5,52 +5,44 @@ package command
 
 import (
 	"github.com/owncloud/ocis/ocis-pkg/config"
-	"github.com/owncloud/ocis/ocis-pkg/version"
+	"github.com/owncloud/ocis/ocis-pkg/shared"
 	"github.com/owncloud/ocis/ocis/pkg/register"
 	"github.com/owncloud/ocis/ocs/pkg/command"
-	svcconfig "github.com/owncloud/ocis/ocs/pkg/config"
 	"github.com/urfave/cli/v2"
 )
 
 // OCSCommand is the entrypoint for the ocs command.
 func OCSCommand(cfg *config.Config) *cli.Command {
+	var globalLog shared.Log
+
 	return &cli.Command{
 		Name:     "ocs",
 		Usage:    "Start ocs server",
 		Category: "Extensions",
+		Before: func(ctx *cli.Context) error {
+			if err := ParseConfig(ctx, cfg); err != nil {
+				return err
+			}
+
+			globalLog = cfg.Log
+
+			return nil
+		},
+		Action: func(c *cli.Context) error {
+			// if accounts logging is empty in ocis.yaml
+			if (cfg.OCS.Log == shared.Log{}) && (globalLog != shared.Log{}) {
+				// we can safely inherit the global logging values.
+				cfg.OCS.Log = globalLog
+			}
+			origCmd := command.Server(cfg.OCS)
+			return handleOriginalAction(c, origCmd)
+		},
 		Subcommands: []*cli.Command{
 			command.PrintVersion(cfg.OCS),
 		},
-		Before: func(ctx *cli.Context) error {
-			return ParseConfig(ctx, cfg)
-		},
-		Action: func(c *cli.Context) error {
-			origCmd := command.Server(configureOCS(cfg))
-			return handleOriginalAction(c, origCmd)
-		},
 	}
 }
 
-func configureOCS(cfg *config.Config) *svcconfig.Config {
-	cfg.OCS.Log.Level = cfg.Log.Level
-	cfg.OCS.Log.Pretty = cfg.Log.Pretty
-	cfg.OCS.Log.Color = cfg.Log.Color
-	cfg.OCS.Service.Version = version.String
-
-	if cfg.Tracing.Enabled {
-		cfg.OCS.Tracing.Enabled = cfg.Tracing.Enabled
-		cfg.OCS.Tracing.Type = cfg.Tracing.Type
-		cfg.OCS.Tracing.Endpoint = cfg.Tracing.Endpoint
-		cfg.OCS.Tracing.Collector = cfg.Tracing.Collector
-	}
-
-	if cfg.TokenManager.JWTSecret != "" {
-		cfg.OCS.TokenManager.JWTSecret = cfg.TokenManager.JWTSecret
-	}
-
-	return cfg.OCS
-}
-
 func init() {
 	register.AddCommand(OCSCommand)
 }