sevenhill/QSfera
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

rename APP_PROVIDER_BASIC_* environment variables, followup of #2811

Browse Source
This commit is contained in:
Willy Kloucek
2021-11-24 15:17:33 +01:00
parent 38d670f1b3
commit 0fc2efadf2
2 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
changelog/unreleased/change-rename-app-provider-env.md
+8
View File
@@ -0,0 +1,8 @@
Change: Rename `APP_PROVIDER_BASIC_*` environment variables
We've renamed the `APP_PROVIDER_BASIC_*` to `APP_PROVIDER_*` since
the `_BASIC_` part is a copy and paste error. Now all app provider
environment variables are consistently starting with `APP_PROVIDER_*`.
https://github.com/owncloud/ocis/pull/2812
https://github.com/owncloud/ocis/pull/2811
storage/pkg/config/config.go
+4 -4
View File
@@ -1082,19 +1082,19 @@ func structMappings(cfg *Config) []shared.EnvBinding {
// app provider // app provider
{ {
EnvVars: []string{"APP_PROVIDER_DEBUG_ADDR", "APP_PROVIDER_BASIC_DEBUG_ADDR"}, // TODO: remove env containing _BASIC_ EnvVars: []string{"APP_PROVIDER_DEBUG_ADDR"},
Destination: &cfg.Reva.AppProvider.DebugAddr, Destination: &cfg.Reva.AppProvider.DebugAddr,
}, },
{ {
EnvVars: []string{"APP_PROVIDER_GRPC_NETWORK", "APP_PROVIDER_BASIC_GRPC_NETWORK"}, // TODO: remove env containing _BASIC_ EnvVars: []string{"APP_PROVIDER_GRPC_NETWORK"},
Destination: &cfg.Reva.AppProvider.GRPCNetwork, Destination: &cfg.Reva.AppProvider.GRPCNetwork,
}, },
{ {
EnvVars: []string{"APP_PROVIDER_GRPC_ADDR", "APP_PROVIDER_BASIC_GRPC_ADDR"}, // TODO: remove env containing _BASIC_ EnvVars: []string{"APP_PROVIDER_GRPC_ADDR"},
Destination: &cfg.Reva.AppProvider.GRPCAddr, Destination: &cfg.Reva.AppProvider.GRPCAddr,
}, },
{ {
EnvVars: []string{"APP_PROVIDER_EXTERNAL_ADDR", "APP_PROVIDER_BASIC_EXTERNAL_ADDR"}, // TODO: remove env containing _BASIC_ EnvVars: []string{"APP_PROVIDER_EXTERNAL_ADDR"},
Destination: &cfg.Reva.AppProvider.ExternalAddr, Destination: &cfg.Reva.AppProvider.ExternalAddr,
}, },
{ {