This is an automated email from the ASF dual-hosted git repository.

ofuks pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/incubator-dlab.git


The following commit(s) were added to refs/heads/develop by this push:
     new 3402cc8  [DLAB-1935] Audit message (#866)
3402cc8 is described below

commit 3402cc8a12e9ca9d955c46b01acf90741066293a
Author: Pavel Papou <ppapo...@gmail.com>
AuthorDate: Tue Aug 25 09:33:47 2020 -0400

    [DLAB-1935] Audit message (#866)
    
    [DLAB-1935] Audit message
---
 .../com/epam/dlab/backendapi/resources/BucketResource.java | 14 ++++++++------
 .../dlab/backendapi/service/impl/ProjectServiceImpl.java   | 11 +++++++++--
 2 files changed, 17 insertions(+), 8 deletions(-)

diff --git 
a/services/self-service/src/main/java/com/epam/dlab/backendapi/resources/BucketResource.java
 
b/services/self-service/src/main/java/com/epam/dlab/backendapi/resources/BucketResource.java
index bbd1b02..a23e1cc 100644
--- 
a/services/self-service/src/main/java/com/epam/dlab/backendapi/resources/BucketResource.java
+++ 
b/services/self-service/src/main/java/com/epam/dlab/backendapi/resources/BucketResource.java
@@ -52,8 +52,10 @@ import java.nio.file.Paths;
 @Path("/bucket")
 @Slf4j
 public class BucketResource {
-    private static final String AUDIT_UPLOAD_FOLDER_MESSAGE = "Folder: %s";
-    private static final String AUDIT_MESSAGE = "File: %s";
+    private static final String AUDIT_FOLDER_UPLOAD_MESSAGE = "Upload folder: 
%s";
+    private static final String AUDIT_FILE_UPLOAD_MESSAGE = "Upload file: %s";
+    private static final String AUDIT_FILE_DOWNLOAD_MESSAGE = "Download file: 
%s";
+    private static final String AUDIT_FILE_DELETE_MESSAGE = "Delete file: %s";
     private static final String OBJECT_FORM_FIELD = "object";
     private static final String BUCKET_FORM_FIELD = "bucket";
     private static final String ENDPOINT_FORM_FIELD = "endpoint";
@@ -93,7 +95,7 @@ public class BucketResource {
     @Produces(MediaType.APPLICATION_JSON)
     @RolesAllowed("/api/bucket/upload")
     public Response uploadFolder(@Auth UserInfo userInfo, @Valid 
FolderUploadDTO dto) {
-        bucketService.uploadFolder(userInfo, dto.getBucket(), dto.getFolder(), 
dto.getEndpoint(), String.format(AUDIT_UPLOAD_FOLDER_MESSAGE, dto.getFolder()));
+        bucketService.uploadFolder(userInfo, dto.getBucket(), dto.getFolder(), 
dto.getEndpoint(), String.format(AUDIT_FOLDER_UPLOAD_MESSAGE, dto.getFolder()));
         return Response.ok().build();
     }
 
@@ -106,7 +108,7 @@ public class BucketResource {
                                    @PathParam("bucket") String bucket,
                                    @PathParam("object") String object,
                                    @PathParam("endpoint") String endpoint) {
-        bucketService.downloadObject(userInfo, bucket, object, endpoint, resp, 
String.format(AUDIT_MESSAGE, object));
+        bucketService.downloadObject(userInfo, bucket, object, endpoint, resp, 
String.format(AUDIT_FILE_DOWNLOAD_MESSAGE, object));
         return Response.ok()
                 .header(HttpHeaders.CONTENT_DISPOSITION, "attachment; 
filename=\"" + Paths.get(object).getFileName() + "\"")
                 .build();
@@ -119,7 +121,7 @@ public class BucketResource {
     @RolesAllowed("/api/bucket/delete")
     public Response deleteObject(@Auth UserInfo userInfo, @Valid 
BucketDeleteDTO bucketDto) {
         final String listOfDeletedObject = String.join(", ", 
bucketDto.getObjects());
-        bucketService.deleteObjects(userInfo, bucketDto.getBucket(), 
bucketDto.getObjects(), bucketDto.getEndpoint(), String.format(AUDIT_MESSAGE, 
listOfDeletedObject));
+        bucketService.deleteObjects(userInfo, bucketDto.getBucket(), 
bucketDto.getObjects(), bucketDto.getEndpoint(), 
String.format(AUDIT_FILE_DELETE_MESSAGE, listOfDeletedObject));
         return Response.ok().build();
     }
 
@@ -146,7 +148,7 @@ public class BucketResource {
                             fileSize = 
Long.parseLong(Streams.asString(stream));
                         }
                     } else {
-                        bucketService.uploadObject(userInfo, bucket, object, 
endpoint, stream, item.getContentType(), fileSize, String.format(AUDIT_MESSAGE, 
object));
+                        bucketService.uploadObject(userInfo, bucket, object, 
endpoint, stream, item.getContentType(), fileSize, 
String.format(AUDIT_FILE_UPLOAD_MESSAGE, object));
                     }
                 } catch (Exception e) {
                     log.error("Cannot upload object {} to bucket {}. {}", 
object, bucket, e.getMessage(), e);
diff --git 
a/services/self-service/src/main/java/com/epam/dlab/backendapi/service/impl/ProjectServiceImpl.java
 
b/services/self-service/src/main/java/com/epam/dlab/backendapi/service/impl/ProjectServiceImpl.java
index 79ee3f1..7e902fc 100644
--- 
a/services/self-service/src/main/java/com/epam/dlab/backendapi/service/impl/ProjectServiceImpl.java
+++ 
b/services/self-service/src/main/java/com/epam/dlab/backendapi/service/impl/ProjectServiceImpl.java
@@ -79,11 +79,13 @@ public class ProjectServiceImpl implements ProjectService {
        private static final String STOP_PRJ_API = 
"infrastructure/project/stop";
        private static final String STOP_ACTION = "stop";
        private static final String TERMINATE_ACTION = "terminate";
+       private static final String TOTAL_BUDGET_PERIOD = "Total";
+       private static final String MONTHLY_BUDGET_PERIOD = "Monthly";
 
        private static final String AUDIT_ADD_ENDPOINT = "Add endpoint(s): 
%s\n";
        private static final String AUDIT_ADD_GROUP = "Add group(s): %s\n";
        private static final String AUDIT_REMOVE_GROUP = "Remove group(s): 
%s\n";
-       private static final String AUDIT_UPDATE_BUDGET = "Update quota: 
%d->%d\nUpdate period: %b";
+       private static final String AUDIT_UPDATE_BUDGET = "Update quota: 
%d->%d\nUpdate period: %s->%s";
        private static final String AUDIT_ADD_EDGE_NODE = "Create edge node for 
endpoint %s, requested in project %s";
 
        private final ProjectDAO projectDAO;
@@ -351,7 +353,12 @@ public class ProjectServiceImpl implements ProjectService {
                Boolean monthlyBudget = 
Optional.ofNullable(projectDTO.getBudget())
                                .map(BudgetDTO::isMonthlyBudget)
                                .orElse(null);
-               return String.format(AUDIT_UPDATE_BUDGET, value, 
p.getBudget().getValue(), monthlyBudget);
+               return String.format(AUDIT_UPDATE_BUDGET, value, 
p.getBudget().getValue(),
+                               representBudgetType(monthlyBudget), 
representBudgetType(p.getBudget().isMonthlyBudget()));
+       }
+
+    private String representBudgetType(Boolean isMonthlyPeriod) {
+               return (isMonthlyPeriod) ? MONTHLY_BUDGET_PERIOD : 
TOTAL_BUDGET_PERIOD;
        }
 
        private List<ProjectEndpointDTO> getProjectEndpointDTOS(List<String> 
endpoints, @Project String name) {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@dlab.apache.org
For additional commands, e-mail: commits-h...@dlab.apache.org

Reply via email to