fix: password not saved correctly when editing asset
This commit is contained in:
+11
-1
@@ -212,7 +212,7 @@ func (assetApi AssetApi) AssetGetEndpoint(c echo.Context) (err error) {
|
||||
id := c.Param("id")
|
||||
|
||||
var item model.Asset
|
||||
if item, err = service.AssetService.FindByIdAndDecrypt(context.TODO(), id); err != nil {
|
||||
if item, err = repository.AssetRepository.FindById(context.TODO(), id); err != nil {
|
||||
return err
|
||||
}
|
||||
attributeMap, err := repository.AssetRepository.FindAssetAttrMapByAssetId(context.TODO(), id)
|
||||
@@ -224,6 +224,16 @@ func (assetApi AssetApi) AssetGetEndpoint(c echo.Context) (err error) {
|
||||
itemMap[key] = attributeMap[key]
|
||||
}
|
||||
|
||||
if item.Password != "" && item.Password != "-" {
|
||||
itemMap["password"] = ""
|
||||
}
|
||||
if item.PrivateKey != "" && item.PrivateKey != "-" {
|
||||
itemMap["privateKey"] = ""
|
||||
}
|
||||
if item.Passphrase != "" && item.Passphrase != "-" {
|
||||
itemMap["passphrase"] = ""
|
||||
}
|
||||
|
||||
return Success(c, itemMap)
|
||||
}
|
||||
|
||||
|
||||
+11
-3
@@ -248,6 +248,11 @@ func (s assetService) UpdateById(id string, m maps.Map) error {
|
||||
return err
|
||||
}
|
||||
|
||||
existingAsset, err := repository.AssetRepository.FindById(context.TODO(), id)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
switch item.AccountType {
|
||||
case "credential":
|
||||
item.Username = "-"
|
||||
@@ -261,9 +266,9 @@ func (s assetService) UpdateById(id string, m maps.Map) error {
|
||||
item.Username = "-"
|
||||
}
|
||||
if len(item.Passphrase) == 0 {
|
||||
item.Passphrase = "-"
|
||||
item.Passphrase = existingAsset.Passphrase
|
||||
}
|
||||
case "custom":
|
||||
case "password":
|
||||
item.PrivateKey = "-"
|
||||
item.Passphrase = "-"
|
||||
item.CredentialId = "-"
|
||||
@@ -271,7 +276,8 @@ func (s assetService) UpdateById(id string, m maps.Map) error {
|
||||
item.Username = "-"
|
||||
}
|
||||
if len(item.Password) == 0 {
|
||||
item.Password = "-"
|
||||
item.Password = existingAsset.Password
|
||||
item.Encrypted = existingAsset.Encrypted
|
||||
}
|
||||
}
|
||||
|
||||
@@ -287,9 +293,11 @@ func (s assetService) UpdateById(id string, m maps.Map) error {
|
||||
item.AccessGatewayId = "-"
|
||||
}
|
||||
|
||||
if item.Password != "" && item.Password != "-" && !item.Encrypted {
|
||||
if err := s.Encrypt(&item, config.GlobalCfg.EncryptionPassword); err != nil {
|
||||
return err
|
||||
}
|
||||
}
|
||||
|
||||
return s.Transaction(context.Background(), func(ctx context.Context) error {
|
||||
if err := repository.AssetRepository.UpdateById(ctx, &item, id); err != nil {
|
||||
|
||||
Reference in New Issue
Block a user