Merge remote-tracking branch 'origin/main' into fix-auth-errors

Author: asafgabai

apptrust/commands/version/update_app_version_cmd.go

@@ -27,8 +27,6 @@ type updateAppVersionCommand struct {
 }
 
 func (uv *updateAppVersionCommand) Run() error {
-	log.Info("Updating application version:", uv.applicationKey, "version:", uv.version)
-
 	ctx, err := service.NewContext(*uv.serverDetails)
 	if err != nil {
 		log.Error("Failed to create service context:", err)
@@ -41,7 +39,6 @@ func (uv *updateAppVersionCommand) Run() error {
 		return err
 	}
 
-	log.Info("Successfully updated application version:", uv.applicationKey, "version:", uv.version)
 	return nil
 }
 

apptrust/service/applications/application_service.go

@@ -36,6 +36,7 @@ func (as *applicationService) CreateApplication(ctx service.Context, requestBody
 			response.StatusCode, responseBody)
 	}
 
+	log.Info(fmt.Sprintf("Application \"%s\" created successfully.", requestBody.ApplicationKey))
 	log.Output(string(responseBody))
 	return nil
 }
@@ -52,6 +53,7 @@ func (as *applicationService) UpdateApplication(ctx service.Context, requestBody
 			response.StatusCode, responseBody)
 	}
 
+	log.Info(fmt.Sprintf("Application \"%s\" updated successfully.", requestBody.ApplicationKey))
 	log.Output(string(responseBody))
 	return nil
 }
@@ -68,6 +70,6 @@ func (as *applicationService) DeleteApplication(ctx service.Context, application
 			response.StatusCode, responseBody)
 	}
 
-	log.Output("Application deleted successfully")
+	log.Info(fmt.Sprintf("Application \"%s\" deleted successfully.", applicationKey))
 	return nil
 }

apptrust/service/packages/package_service.go

@@ -35,6 +35,7 @@ func (ps *packageService) BindPackage(ctx service.Context, applicationKey string
 			response.StatusCode, responseBody)
 	}
 
+	log.Info("Package bound successfully.")
 	log.Output(string(responseBody))
 	return nil
 }
@@ -51,6 +52,6 @@ func (ps *packageService) UnbindPackage(ctx service.Context, applicationKey, pkg
 			response.StatusCode, responseBody)
 	}
 
-	log.Output("Package unbound successfully")
+	log.Info("Package unbound successfully.")
 	return nil
 }

apptrust/service/versions/version_service.go

@@ -40,6 +40,7 @@ func (vs *versionService) CreateAppVersion(ctx service.Context, request *model.C
 			response.StatusCode, responseBody)
 	}
 
+	log.Info("Application version created successfully.")
 	log.Output(string(responseBody))
 	return nil
 }
@@ -110,6 +111,7 @@ func (vs *versionService) DeleteAppVersion(ctx service.Context, applicationKey,
 			response.StatusCode, responseBody)
 	}
 
+	log.Info("Application version deleted successfully.")
 	return nil
 }
 
@@ -125,5 +127,6 @@ func (vs *versionService) UpdateAppVersion(ctx service.Context, applicationKey s
 			response.StatusCode, responseBody)
 	}
 
+	log.Info("Application version updated successfully.")
 	return nil
 }