Skip to content

Commit

Permalink
Merge pull request #882 from kaleido-io/dxtype
Browse files Browse the repository at this point in the history
Deprecate default value for "ffdx"
  • Loading branch information
shorsher authored Jun 28, 2022
2 parents 30b0c56 + 4677c10 commit 522007c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 0 additions & 1 deletion internal/coreconfig/coreconfig.go
Original file line number Diff line number Diff line change
Expand Up @@ -312,7 +312,6 @@ func setDefaults() {
viper.SetDefault(string(CorsAllowedOrigins), []string{"*"})
viper.SetDefault(string(CorsEnabled), true)
viper.SetDefault(string(CorsMaxAge), 600)
viper.SetDefault(string(DataexchangeType), "ffdx")
viper.SetDefault(string(HistogramsMaxChartRows), 100)
viper.SetDefault(string(DebugPort), -1)
viper.SetDefault(string(DownloadWorkerCount), 10)
Expand Down
4 changes: 4 additions & 0 deletions internal/orchestrator/orchestrator.go
Original file line number Diff line number Diff line change
Expand Up @@ -359,6 +359,10 @@ func (or *orchestrator) initDatabaseCheckPreinit(ctx context.Context) (err error

func (or *orchestrator) initDataExchange(ctx context.Context) (err error) {
dxPlugin := config.GetString(coreconfig.DataexchangeType)
if dxPlugin == "" {
dxPlugin = "ffdx"
log.L(ctx).Warnf("The default value for %s has been deprecated - please explicitly set it to \"ffdx\"", coreconfig.DataexchangeType)
}
if or.dataexchange == nil {
pluginName := dxPlugin
if or.dataexchange, err = dxfactory.GetPlugin(ctx, pluginName); err != nil {
Expand Down

0 comments on commit 522007c

Please sign in to comment.