Skip to content

Commit

Permalink
refactor!: Don't add version to Config Stem (#401)
Browse files Browse the repository at this point in the history
BREAKING CHANGE: Service configuration location in Consul has changed

Version is now in the default config stem

closes #400

Signed-off-by: Leonard Goodell <[email protected]>
  • Loading branch information
Lenny Goodell authored Dec 13, 2022
1 parent 608b320 commit 6cf9e04
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions bootstrap/config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,7 @@ import (
)

const (
writableKey = "/Writable"
ConfigVersion = "2.0"
writableKey = "/Writable"
)

// UpdatedStream defines the stream type that is notified by ListenForChanges when a configuration update is received.
Expand Down Expand Up @@ -355,7 +354,7 @@ func (cp *Processor) createProviderClient(
}

// Note: Can't use filepath.Join as it uses `\` on Windows which Consul doesn't recognize as a path separator.
providerConfig.BasePath = fmt.Sprintf("%s%s/%s", configStem, ConfigVersion, serviceKey)
providerConfig.BasePath = fmt.Sprintf("%s%s", configStem, serviceKey)
if getAccessToken != nil {
providerConfig.AccessToken, err = getAccessToken()
if err != nil {
Expand Down

0 comments on commit 6cf9e04

Please sign in to comment.