define ocis_url as a shared option in ocis-pkg/config

ocis-pkg/config/config.go

@@ -83,8 +83,9 @@ type Config struct {
 	// Mode is mostly used whenever we need to run an extension. The technical debt this introduces is in regard of
 	// what it does. Supervised (0) loads configuration from a unified config file because of known limitations of Viper; whereas
 	// Unsupervised (1) MUST parse config from all known sources.
-	Mode Mode
-	File string
+	Mode    Mode
+	File    string
+	OcisURL string `mapstructure:"ocis_url"`
 
 	Registry     string       `mapstructure:"registry"`
 	Log          Log          `mapstructure:"log"`

ocis/pkg/command/proxy.go

@@ -18,7 +18,6 @@ func ProxyCommand(cfg *config.Config) *cli.Command {
 		Name:     "proxy",
 		Usage:    "Start proxy server",
 		Category: "Extensions",
-		//Flags:    flagset.ServerWithConfig(cfg.Proxy),
 		Subcommands: []*cli.Command{
 			command.PrintVersion(cfg.Proxy),
 		},
@@ -33,6 +32,7 @@ func ProxyCommand(cfg *config.Config) *cli.Command {
 }
 
 func configureProxy(cfg *config.Config) *svcconfig.Config {
+	cfg.Proxy.OcisURL = cfg.OcisURL
 	cfg.Proxy.Log.Level = cfg.Log.Level
 	cfg.Proxy.Log.Pretty = cfg.Log.Pretty
 	cfg.Proxy.Log.Color = cfg.Log.Color

ocis/pkg/command/root.go

@@ -21,7 +21,7 @@ func Execute() error {
 		Usage:    "ownCloud Infinite Scale Stack",
 		Compiled: version.Compiled(),
 		Before: func(c *cli.Context) error {
-			return ParseConfig(cfg)
+			return ParseConfig(c, cfg)
 		},
 		Authors: []*cli.Author{
 			{
@@ -63,7 +63,7 @@ func NewLogger(cfg *config.Config) log.Logger {
 }
 
 // ParseConfig loads ocis configuration from known paths.
-func ParseConfig(cfg *config.Config) error {
+func ParseConfig(c *cli.Context, cfg *config.Config) error {
 	conf, err := ociscfg.BindSourcesToStructs("ocis", cfg)
 	if err != nil {
 		return err

proxy/pkg/command/server.go

@@ -52,6 +52,14 @@ func Server(cfg *config.Config) *cli.Command {
 				cfg.PreSignedURL.AllowedHTTPMethods = ctx.StringSlice("presignedurl-allow-method")
 			}
 
+			// we need a starting point to compare the default config values to determine if the outcome of ParseConfig
+			// modified a value that should be shared just because it is present in the ocis.yaml file.
+			defaultConfig := config.DefaultConfig()
+
+			if cfg.OcisURL != "" && cfg.OIDC.Issuer == defaultConfig.OIDC.Issuer {
+				cfg.OIDC.Issuer = cfg.OcisURL
+			}
+
 			if err := loadUserAgent(ctx, cfg); err != nil {
 				return err
 			}

proxy/pkg/config/config.go

@@ -105,6 +105,8 @@ type Cache struct {
 
 // Config combines all available configuration parts.
 type Config struct {
+	OcisURL string
+
 	File                  string          `mapstructure:"file"`
 	Log                   Log             `mapstructure:"log"`
 	Debug                 Debug           `mapstructure:"debug"`

proxy/pkg/config/mappings.go

@@ -123,7 +123,7 @@ func structMappings(cfg *Config) []mapping {
 			Destination: &cfg.InsecureBackends,
 		},
 		{
-			EnvVars:     []string{"PROXY_OIDC_ISSUER", "OCIS_URL"}, // PROXY_OIDC_ISSUER takes precedence over OCIS_URL
+			EnvVars:     []string{"PROXY_OIDC_ISSUER"},
 			Destination: &cfg.OIDC.Issuer,
 		},
 		{