remove returns from the commands to prevent silent crash

Signed-off-by: Christian Richter <crichter@owncloud.com>

ocis/pkg/command/accounts.go

@@ -19,7 +19,6 @@ func AccountsCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Accounts.Commons = cfg.Commons
 			return nil

ocis/pkg/command/app-registry.go

@@ -19,7 +19,6 @@ func AppRegistryCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.AppRegistry.Commons = cfg.Commons
 			return nil

ocis/pkg/command/appprovider.go

@@ -19,7 +19,6 @@ func AppProviderCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.AppProvider.Commons = cfg.Commons
 			return nil

ocis/pkg/command/audit.go

@@ -19,7 +19,6 @@ func AuditCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Audit.Commons = cfg.Commons
 			return nil

ocis/pkg/command/auth-basic.go

@@ -19,7 +19,6 @@ func AuthBasicCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.AuthBasic.Commons = cfg.Commons
 			return nil

ocis/pkg/command/auth-bearer.go

@@ -19,7 +19,6 @@ func AuthBearerCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.AuthBearer.Commons = cfg.Commons
 			return nil

ocis/pkg/command/auth-machine.go

@@ -19,7 +19,6 @@ func AuthMachineCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.AuthMachine.Commons = cfg.Commons
 			return nil

ocis/pkg/command/frontend.go

@@ -19,7 +19,6 @@ func FrontendCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Frontend.Commons = cfg.Commons
 			return nil

ocis/pkg/command/gateway.go

@@ -19,7 +19,6 @@ func GatewayCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Gateway.Commons = cfg.Commons
 			return nil

ocis/pkg/command/glauth.go

@@ -19,7 +19,6 @@ func GLAuthCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.GLAuth.Commons = cfg.Commons
 			return nil

ocis/pkg/command/graph.go

@@ -19,7 +19,6 @@ func GraphCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Graph.Commons = cfg.Commons
 			return nil

ocis/pkg/command/graphexplorer.go

@@ -19,7 +19,6 @@ func GraphExplorerCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.GraphExplorer.Commons = cfg.Commons
 			return nil

ocis/pkg/command/groups.go

@@ -19,7 +19,6 @@ func GroupsCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Groups.Commons = cfg.Commons
 			return nil

ocis/pkg/command/idm.go

@@ -19,7 +19,6 @@ func IDMCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.IDM.Commons = cfg.Commons
 			return nil

ocis/pkg/command/idp.go

@@ -19,7 +19,6 @@ func IDPCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.IDP.Commons = cfg.Commons
 			return nil

ocis/pkg/command/natsserver.go

@@ -19,7 +19,6 @@ func NatsCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Nats.Commons = cfg.Commons
 			return nil

ocis/pkg/command/notifications.go

@@ -19,7 +19,6 @@ func NotificationsCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Notifications.Commons = cfg.Commons
 			return nil

ocis/pkg/command/ocdav.go

@@ -19,7 +19,6 @@ func OCDavCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.OCDav.Commons = cfg.Commons
 			return nil

ocis/pkg/command/ocs.go

@@ -19,7 +19,6 @@ func OCSCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.OCS.Commons = cfg.Commons
 			return nil

ocis/pkg/command/proxy.go

@@ -19,7 +19,6 @@ func ProxyCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Proxy.Commons = cfg.Commons
 			return nil

ocis/pkg/command/settings.go

@@ -19,7 +19,6 @@ func SettingsCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Settings.Commons = cfg.Commons
 			return nil

ocis/pkg/command/sharing.go

@@ -19,7 +19,6 @@ func SharingCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Sharing.Commons = cfg.Commons
 			return nil

ocis/pkg/command/storage-metadata.go

@@ -19,7 +19,6 @@ func StorageSystemCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.StorageSystem.Commons = cfg.Commons
 			return nil

ocis/pkg/command/storage-publiclink.go

@@ -19,7 +19,6 @@ func StoragePublicLinkCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.StoragePublicLink.Commons = cfg.Commons
 			return nil

ocis/pkg/command/storage-shares.go

@@ -19,7 +19,6 @@ func StorageSharesCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.StorageShares.Commons = cfg.Commons
 			return nil

ocis/pkg/command/storage-users.go

@@ -19,7 +19,6 @@ func StorageUsersCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.StorageUsers.Commons = cfg.Commons
 			return nil

ocis/pkg/command/store.go

@@ -20,7 +20,6 @@ func StoreCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Store.Commons = cfg.Commons
 			return nil

ocis/pkg/command/thumbnails.go

@@ -19,7 +19,6 @@ func ThumbnailsCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Thumbnails.Commons = cfg.Commons
 			return nil

ocis/pkg/command/users.go

@@ -19,7 +19,6 @@ func UsersCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Users.Commons = cfg.Commons
 			return nil

ocis/pkg/command/version.go

@@ -28,7 +28,6 @@ func VersionCommand(cfg *config.Config) *cli.Command {
 			serviceList, err := reg.ListServices()
 			if err != nil {
 				fmt.Println(fmt.Errorf("could not list services: %v", err))
-				return err
 			}
 
 			var services []*mreg.Service

ocis/pkg/command/web.go

@@ -19,7 +19,6 @@ func WebCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.Web.Commons = cfg.Commons
 			return nil

ocis/pkg/command/webdav.go

@@ -20,7 +20,6 @@ func WebDAVCommand(cfg *config.Config) *cli.Command {
 		Before: func(c *cli.Context) error {
 			if err := parser.ParseConfig(cfg); err != nil {
 				fmt.Printf("%v", err)
-				return err
 			}
 			cfg.WebDAV.Commons = cfg.Commons
 			return nil