Skip to content

Commit c3b36cf

Browse files
committed
fix(store-types): Use embedded store-types json file
Signed-off-by: spbsoluble <[email protected]>
1 parent 6ed9b54 commit c3b36cf

File tree

2 files changed

+26
-15
lines changed

2 files changed

+26
-15
lines changed

cmd/constants.go

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -16,19 +16,20 @@ package cmd
1616
import "fmt"
1717

1818
const (
19-
ColorRed = "\033[31m"
20-
ColorWhite = "\033[37m"
21-
DefaultAPIPath = "KeyfactorAPI"
22-
DefaultConfigFileName = "command_config.json"
23-
FailedAuthMsg = "Login failed!"
24-
SuccessfulAuthMsg = "Login successful!"
25-
XKeyfactorRequestedWith = "APIClient"
26-
XKeyfactorApiVersion = "1"
27-
FlagGitRef = "git-ref"
28-
FlagFromFile = "from-file"
29-
DebugFuncEnter = "entered: %s"
30-
DebugFuncExit = "exiting: %s"
31-
DebugFuncCall = "calling: %s"
19+
ColorRed = "\033[31m"
20+
ColorWhite = "\033[37m"
21+
DefaultAPIPath = "KeyfactorAPI"
22+
DefaultConfigFileName = "command_config.json"
23+
DefaultStoreTypesFileName = "store_types.json"
24+
FailedAuthMsg = "Login failed!"
25+
SuccessfulAuthMsg = "Login successful!"
26+
XKeyfactorRequestedWith = "APIClient"
27+
XKeyfactorApiVersion = "1"
28+
FlagGitRef = "git-ref"
29+
FlagFromFile = "from-file"
30+
DebugFuncEnter = "entered: %s"
31+
DebugFuncExit = "exiting: %s"
32+
DebugFuncCall = "calling: %s"
3233
)
3334

3435
var ProviderTypeChoices = []string{

cmd/storeTypes.go

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -498,7 +498,17 @@ func readStoreTypesConfig(fp string, gitRef string) (map[string]interface{}, err
498498
Msg(fmt.Sprintf(DebugFuncCall, "getStoreTypesInternet"))
499499
sTypes, stErr := getStoreTypesInternet(gitRef)
500500
if stErr != nil {
501-
log.Error().Err(stErr).Msg("unable to read store types from internet, attempting to reference embedded definitions")
501+
log.Warn().
502+
Err(stErr).
503+
Msg("unable to read store types from internet, attempting to reference embedded definitions")
504+
if err := json.Unmarshal(EmbeddedStoreTypesJSON, &sTypes); err != nil {
505+
log.Error().Err(err).Msg("unable to unmarshal embedded store type definitions")
506+
return nil, err
507+
}
508+
} else if sTypes == nil || len(sTypes) == 0 {
509+
log.Warn().Err(fmt.Errorf("empty store valid store types list")).Msg(
510+
"0 store types found from internet, attempting to reference embedded definitions",
511+
)
502512
if err := json.Unmarshal(EmbeddedStoreTypesJSON, &sTypes); err != nil {
503513
log.Error().Err(err).Msg("unable to unmarshal embedded store type definitions")
504514
return nil, err
@@ -509,7 +519,7 @@ func readStoreTypesConfig(fp string, gitRef string) (map[string]interface{}, err
509519
var err error
510520
if sTypes == nil {
511521
if fp == "" {
512-
fp = "store_types.json"
522+
fp = DefaultStoreTypesFileName
513523
}
514524
content, err = os.ReadFile(fp)
515525
if err != nil {

0 commit comments

Comments
 (0)