Skip to content

Commit 07248af

Browse files
config-file-save
1 parent c2835af commit 07248af

File tree

6 files changed

+23
-21
lines changed

6 files changed

+23
-21
lines changed

internal/commands/auth.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,9 @@ func validLogin() func(cmd *cobra.Command, args []string) error {
121121
return func(cmd *cobra.Command, args []string) error {
122122
defer func() {
123123
logger.PrintIfVerbose("Calling GetUniqueId func")
124-
uniqueId := wrappers.GetUniqueId()
125-
if uniqueId != "" {
126-
logger.PrintIfVerbose("Set unique id: " + uniqueId)
124+
uniqueID := wrappers.GetUniqueID()
125+
if uniqueID != "" {
126+
logger.PrintIfVerbose("Set unique id: " + uniqueID)
127127
}
128128
}()
129129
clientID := viper.GetString(params.AccessKeyIDConfigKey)

internal/commands/telemetry.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ func runTelemetryAI(telemetryWrapper wrappers.TelemetryWrapper) func(*cobra.Comm
5959
scanType, _ := cmd.Flags().GetString("scan-type")
6060
status, _ := cmd.Flags().GetString("status")
6161
totalCount, _ := cmd.Flags().GetInt("total-count")
62-
uniqueId := wrappers.GetUniqueId()
62+
uniqueId := wrappers.GetUniqueID()
6363
logger.PrintIfVerbose("unique id: " + uniqueId)
6464
err := telemetryWrapper.SendAIDataToLog(&wrappers.DataForAITelemetry{
6565
AIProvider: aiProvider,
@@ -71,7 +71,7 @@ func runTelemetryAI(telemetryWrapper wrappers.TelemetryWrapper) func(*cobra.Comm
7171
ScanType: scanType,
7272
Status: status,
7373
TotalCount: totalCount,
74-
UniqueId: uniqueId,
74+
UniqueID: uniqueId,
7575
})
7676

7777
if err != nil {

internal/params/envs.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,5 +80,5 @@ const (
8080
RiskManagementPathEnv = "CX_RISK_MANAGEMENT_PATH"
8181
ConfigFilePathEnv = "CX_CONFIG_FILE_PATH"
8282
RealtimeScannerPathEnv = "CX_REALTIME_SCANNER_PATH"
83-
UniqueIdEnv = "CX_UNIQUE_ID"
83+
UniqueIDEnv = "CX_UNIQUE_ID"
8484
)

internal/params/keys.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,5 +79,5 @@ var (
7979
RiskManagementPathKey = strings.ToLower(RiskManagementPathEnv)
8080
ConfigFilePathKey = strings.ToLower(ConfigFilePathEnv)
8181
RealtimeScannerPathKey = strings.ToLower(RealtimeScannerPathEnv)
82-
UniqueIdConfigKey = strings.ToLower(UniqueIdEnv)
82+
UniqueIDConfigKey = strings.ToLower(UniqueIDEnv)
8383
)

internal/wrappers/client.go

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ import (
2727
"github.com/spf13/viper"
2828

2929
commonParams "github.com/checkmarx/ast-cli/internal/params"
30+
"github.com/checkmarx/ast-cli/internal/wrappers/configuration"
3031
"github.com/checkmarx/ast-cli/internal/wrappers/kerberos"
3132
"github.com/checkmarx/ast-cli/internal/wrappers/ntlm"
3233
)
@@ -137,10 +138,10 @@ func setAgentNameAndOrigin(req *http.Request, isAuth bool) {
137138
logger.PrintIfVerbose("getting unique id")
138139

139140
if !isAuth {
140-
uniqueId := GetUniqueId()
141-
if uniqueId != "" {
142-
req.Header.Set("UniqueId", uniqueId)
143-
logger.PrintIfVerbose("unique id: " + uniqueId)
141+
uniqueID := GetUniqueID()
142+
if uniqueID != "" {
143+
req.Header.Set("UniqueId", uniqueID)
144+
logger.PrintIfVerbose("unique id: " + uniqueID)
144145
}
145146
}
146147
}
@@ -984,7 +985,7 @@ func extractAZPFromToken(astToken string) (string, error) {
984985
return azp, nil
985986
}
986987

987-
func GetUniqueId() string {
988+
func GetUniqueID() string {
988989
isAllowed := false
989990
accessToken, err := GetAccessToken()
990991
if err != nil {
@@ -1013,23 +1014,24 @@ func GetUniqueId() string {
10131014
logger.PrintIfVerbose("User does not not have permission to standalone dev asists feature")
10141015
return ""
10151016
}
1016-
uniqueId := viper.GetString(commonParams.UniqueIdConfigKey)
1017-
if uniqueId != "" {
1018-
return uniqueId
1017+
uniqueID := viper.GetString(commonParams.UniqueIDConfigKey)
1018+
if uniqueID != "" {
1019+
return uniqueID
10191020
}
10201021
logger.PrintIfVerbose("Generating new unique id")
10211022
currentUser, err := user.Current()
10221023
if err != nil {
10231024
logger.PrintIfVerbose("Failed to get user: " + err.Error())
10241025
return ""
10251026
}
1026-
uniqueId = uuid.New().String() + currentUser.Username
1027-
logger.PrintIfVerbose("Unique id: " + uniqueId)
1028-
viper.Set(commonParams.UniqueIdConfigKey, uniqueId)
1029-
err = viper.WriteConfig()
1027+
uniqueID = uuid.New().String() + "_" + currentUser.Username
1028+
logger.PrintIfVerbose("Unique id: " + uniqueID)
1029+
viper.Set(commonParams.UniqueIDConfigKey, uniqueID)
1030+
configFilePath, _ := configuration.GetConfigFilePath()
1031+
err = configuration.SafeWriteSingleConfigKeyString(configFilePath, commonParams.UniqueIDConfigKey, uniqueID)
10301032
if err != nil {
10311033
logger.PrintIfVerbose("Failed to write config: " + err.Error())
10321034
return ""
10331035
}
1034-
return uniqueId
1036+
return uniqueID
10351037
}

internal/wrappers/telemetry.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ type DataForAITelemetry struct {
1010
ScanType string `json:"scanType"`
1111
Status string `json:"status"`
1212
TotalCount int `json:"totalCount"`
13-
UniqueId string `json:"uniqueId"`
13+
UniqueID string `json:"uniqueId"`
1414
}
1515

1616
type TelemetryWrapper interface {

0 commit comments

Comments
 (0)