Merge pull request #278 from Keyfactor/release-1.8

Release 1.8.2

Author: spbsoluble

.github/workflows/update-stores.yml

@@ -108,7 +108,7 @@ jobs:
 
       - name: Save Store Types JSON Artifact
         if: success()
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: store-types
           path: |
@@ -117,14 +117,14 @@ jobs:
 
       - name: Save Invalid Store Types JSON Artifact
         if: success()
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: invalid-repos
           path: ./tools/store-type-merge/invalid_repos.json
 
       - name: Save logs directory
         if: success()
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: logs
           path: ./tools/store-type-merge/log

CHANGELOG.md

@@ -1,3 +1,20 @@
+# v1.8.2
+
+## Fixes
+
+### CLI
+
+- Silence logging of initial `store-type` fetch
+- `store-types`: Update internal definitions to latest from Keyfactor GitHub repos.
+- `version` update internal version to `1.8.2`
+
+### Chores
+
+- `docs`: Add `migration` auto docs
+- `docs`: Refresh auto docs
+- `deps`: Update all deps to latest versions
+
+
 # v1.8.1
 
 ## Fixes

cmd/helpers.go

@@ -186,7 +186,10 @@ func informDebug(debugFlag bool) {
 	if debugModeEnabled {
 		//zerolog.SetGlobalLevel(zerolog.InfoLevel)
 		zerolog.SetGlobalLevel(zerolog.DebugLevel)
+		return
 	}
+	// Else no logs should be emitted
+	zerolog.SetGlobalLevel(zerolog.Disabled)
 }
 
 func initLogger() {
@@ -359,7 +362,7 @@ func outputResult(result interface{}, format string) {
 		fmt.Println(fmt.Sprintf("%s", result))
 	}
 	log.Debug().
-		Interface("result", result).
+		Str("result", fmt.Sprintf("%s", result)).
 		Str("format", format).
 		Msg(fmt.Sprintf("%s outputResult", DebugFuncExit))
 }

cmd/storeTypes.go

@@ -636,10 +636,11 @@ func readStoreTypesConfig(fp, gitRef string, gitRepo string, offline bool) (map[
 }
 
 func init() {
-	offline = true    // temporarily set to true as it runs before the flag is set
-	debugFlag = false // temporarily set to false as it runs before the flag is set
+	offline = true // temporarily set to true as it runs before the flag is set
 	var gitRef string
 	var gitRepo string
+
+	informDebug(false) // Do not output debug information for initial setup of store-types command
 	validTypesString := strings.Join(getValidStoreTypes("", DefaultGitRef, DefaultGitRepo), ", ")
 	offline = false //revert this so that flag is not set to true by default
 	RootCmd.AddCommand(storeTypesCmd)