Refactor constants and enums

RADAR-base · Jun 3, 2024 · e22b66d · e22b66d
1 parent e7a6a43
commit e22b66d
Show file tree

Hide file tree

Showing 14 changed files with 146 additions and 138 deletions.
diff --git a/src/main/java/org/radarbase/appserver/config/AuthConfig.java b/src/main/java/org/radarbase/appserver/config/AuthConfig.java
@@ -64,24 +64,4 @@ AuthAspect getAuthAspect(
  return new AuthAspect<>(authValidator, authorization);
  }
 
- public interface AuthEntities {
- String MEASUREMENT = "MEASUREMENT";
- String PROJECT = "PROJECT";
- String SUBJECT = "SUBJECT";
- String SOURCE = "SOURCE";
- }
-
- public interface AuthPermissions {
- String READ = "READ";
- String CREATE = "CREATE";
- String UPDATE = "UPDATE";
- String UPLOAD = "UPLOAD";
- }
-
- public enum AuthPermission {
- READ,
- CREATE,
- UPDATE,
- UPLOAD
- }
 }
diff --git a/src/main/java/org/radarbase/appserver/config/AuthEntities.java b/src/main/java/org/radarbase/appserver/config/AuthEntities.java
@@ -0,0 +1,18 @@
+package org.radarbase.appserver.config;
+
+public enum AuthEntities {
+ MEASUREMENT(Constants.MEASUREMENT),
+ PROJECT(Constants.PROJECT),
+ SUBJECT(Constants.SUBJECT),
+ SOURCE(Constants.SOURCE);
+
+ AuthEntities(String entityValue) {
+ }
+
+ public static class Constants {
+ public static final String MEASUREMENT = "MEASUREMENT";
+ public static final String PROJECT = "PROJECT";
+ public static final String SUBJECT = "SUBJECT";
+ public static final String SOURCE = "SOURCE";
+ }
+}
diff --git a/src/main/java/org/radarbase/appserver/config/AuthPermission.java b/src/main/java/org/radarbase/appserver/config/AuthPermission.java
@@ -0,0 +1,18 @@
+package org.radarbase.appserver.config;
+
+public enum AuthPermission {
+ READ(Constants.READ),
+ CREATE(Constants.CREATE),
+ UPDATE(Constants.UPDATE),
+ UPLOAD(Constants.UPLOAD);
+
+ AuthPermission(String permissionValue) {
+ }
+
+ public static class Constants {
+ public static final String READ = "READ";
+ public static final String CREATE = "CREATE";
+ public static final String UPDATE = "UPDATE";
+ public static final String UPLOAD = "UPLOAD";
+ }
+}
diff --git a/src/main/java/org/radarbase/appserver/config/PreAuthPermissionEvaluator.java b/src/main/java/org/radarbase/appserver/config/PreAuthPermissionEvaluator.java
@@ -1,6 +1,5 @@
 package org.radarbase.appserver.config;
 
-import org.radarbase.appserver.config.AuthConfig.AuthPermission;
 import org.springframework.security.access.PermissionEvaluator;
 import org.springframework.security.core.Authentication;
 

diff --git a/src/main/java/org/radarbase/appserver/controller/FcmDataMessageController.java b/src/main/java/org/radarbase/appserver/controller/FcmDataMessageController.java
@@ -25,8 +25,8 @@
 import java.net.URISyntaxException;
 import java.time.LocalDateTime;
 import jakarta.validation.Valid;
-import org.radarbase.appserver.config.AuthConfig.AuthEntities;
-import org.radarbase.appserver.config.AuthConfig.AuthPermissions;
+import org.radarbase.appserver.config.AuthEntities;
+import org.radarbase.appserver.config.AuthPermission;
 import org.radarbase.appserver.dto.fcm.FcmDataMessageDto;
 import org.radarbase.appserver.dto.fcm.FcmDataMessages;
 import org.radarbase.appserver.service.FcmDataMessageService;
@@ -59,20 +59,20 @@ public FcmDataMessageController(FcmDataMessageService dataMessageService) {
  }
 
  @GetMapping("/" + PathsUtil.MESSAGING_DATA_PATH)
- @Authorized(permission = AuthPermissions.READ, entity = AuthEntities.PROJECT)
+ @Authorized(permission = AuthPermission.Constants.READ, entity = AuthEntities.Constants.PROJECT)
  public ResponseEntity<FcmDataMessages> getAllDataMessages() {
  return ResponseEntity.ok(this.dataMessageService.getAllDataMessages());
  }
 
- @Authorized(permission = AuthPermissions.READ, entity = AuthEntities.SUBJECT)
+ @Authorized(permission = AuthPermission.Constants.READ, entity = AuthEntities.Constants.SUBJECT)
  @GetMapping("/" + PathsUtil.MESSAGING_DATA_PATH + "/{id}")
  public ResponseEntity<FcmDataMessageDto> getDataMessageUsingId(@Valid @PathVariable Long id) {
  return ResponseEntity.ok(this.dataMessageService.getDataMessageById(id));
  }
 
  // TODO: get notifications/data messages based on other params. Maybe use projections ?
  @GetMapping("/" + PathsUtil.MESSAGING_DATA_PATH + "/filtered")
- @Authorized(permission = AuthPermissions.READ, entity = AuthEntities.PROJECT)
+ @Authorized(permission = AuthPermission.Constants.READ, entity = AuthEntities.Constants.PROJECT)
  public ResponseEntity<FcmDataMessages> getFilteredDataMessages(
  @Valid @RequestParam(value = "type", required = false) String type,
  @Valid @RequestParam(value = "delivered", required = false) boolean delivered,
@@ -86,8 +86,8 @@ public ResponseEntity<FcmDataMessages> getFilteredDataMessages(
  }
 
  @Authorized(
- permission = AuthPermissions.READ,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.READ,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @GetMapping(
  value =
@@ -108,8 +108,8 @@ public ResponseEntity<FcmDataMessages> getDataMessagesUsingProjectIdAndSubjectId
  }
 
  @Authorized(
- permission = AuthPermissions.READ,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.READ,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.PROJECT)
  @GetMapping(
  "/"
@@ -124,8 +124,8 @@ public ResponseEntity<FcmDataMessages> getDataMessagesUsingProjectId(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  "/"
@@ -151,8 +151,8 @@ public ResponseEntity<FcmDataMessageDto> addSingleDataMessage(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  "/"
@@ -175,8 +175,8 @@ public ResponseEntity<FcmDataMessages> addBatchDataMessages(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PutMapping(
  "/"
@@ -199,8 +199,8 @@ public ResponseEntity<FcmDataMessageDto> updateDataMessage(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @DeleteMapping(
  "/"
@@ -223,8 +223,8 @@ public ResponseEntity deleteDataMessagesForUser(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @DeleteMapping(
  "/"

diff --git a/src/main/java/org/radarbase/appserver/controller/FcmNotificationController.java b/src/main/java/org/radarbase/appserver/controller/FcmNotificationController.java
@@ -25,8 +25,8 @@
 import java.net.URISyntaxException;
 import java.time.LocalDateTime;
 import jakarta.validation.Valid;
-import org.radarbase.appserver.config.AuthConfig.AuthEntities;
-import org.radarbase.appserver.config.AuthConfig.AuthPermissions;
+import org.radarbase.appserver.config.AuthEntities;
+import org.radarbase.appserver.config.AuthPermission;
 import org.radarbase.appserver.dto.fcm.FcmNotificationDto;
 import org.radarbase.appserver.dto.fcm.FcmNotifications;
 import org.radarbase.appserver.service.FcmNotificationService;
@@ -58,19 +58,19 @@ public FcmNotificationController(FcmNotificationService notificationService) {
  this.notificationService = notificationService;
  }
 
- @Authorized(permission = AuthPermissions.READ, entity = AuthEntities.PROJECT)
+ @Authorized(permission = AuthPermission.Constants.READ, entity = AuthEntities.Constants.PROJECT)
  @GetMapping("/" + PathsUtil.MESSAGING_NOTIFICATION_PATH)
  public ResponseEntity<FcmNotifications> getAllNotifications() {
  return ResponseEntity.ok(this.notificationService.getAllNotifications());
  }
 
- @Authorized(permission = AuthPermissions.UPDATE, entity = AuthEntities.SUBJECT)
+ @Authorized(permission = AuthPermission.Constants.UPDATE, entity = AuthEntities.Constants.SUBJECT)
  @GetMapping("/" + PathsUtil.MESSAGING_NOTIFICATION_PATH + "/{id}")
  public ResponseEntity<FcmNotificationDto> getNotificationUsingId(@Valid @PathVariable Long id) {
  return ResponseEntity.ok(this.notificationService.getNotificationById(id));
  }
  // TODO: get notifications based on other params. Maybe use projections ?
- @Authorized(permission = AuthPermissions.READ, entity = AuthEntities.PROJECT)
+ @Authorized(permission = AuthPermission.Constants.READ, entity = AuthEntities.Constants.PROJECT)
  @GetMapping("/" + PathsUtil.MESSAGING_NOTIFICATION_PATH + "/filtered")
  public ResponseEntity<FcmNotifications> getFilteredNotifications(
  @Valid @RequestParam(value = "type", required = false) String type,
@@ -85,8 +85,8 @@ public ResponseEntity<FcmNotifications> getFilteredNotifications(
  }
 
  @Authorized(
- permission = AuthPermissions.READ,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.READ,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @GetMapping(
  value =
@@ -107,8 +107,8 @@ public ResponseEntity<FcmNotifications> getNotificationsUsingProjectIdAndSubject
  }
 
  @Authorized(
- permission = AuthPermissions.READ,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.READ,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.PROJECT)
  @GetMapping(
  "/"
@@ -123,8 +123,8 @@ public ResponseEntity<FcmNotifications> getNotificationsUsingProjectId(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  "/"
@@ -151,8 +151,8 @@ public ResponseEntity<FcmNotificationDto> addSingleNotification(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  "/"
@@ -175,8 +175,8 @@ public ResponseEntity<FcmNotifications> scheduleUserNotifications(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  "/"
@@ -202,8 +202,8 @@ public ResponseEntity<FcmNotificationDto> scheduleUserNotification(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  "/"
@@ -227,8 +227,8 @@ public ResponseEntity<FcmNotifications> addBatchNotifications(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PutMapping(
  "/"
@@ -251,8 +251,8 @@ public ResponseEntity<FcmNotificationDto> updateNotification(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @DeleteMapping(
  "/"
@@ -275,8 +275,8 @@ public ResponseEntity deleteNotificationsForUser(
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @DeleteMapping(
  "/"
@@ -300,8 +300,8 @@ public ResponseEntity deleteNotificationUsingProjectIdAndSubjectIdAndNotificatio
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermission.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @DeleteMapping(
  "/"

diff --git a/src/main/java/org/radarbase/appserver/controller/GithubEndpoint.java b/src/main/java/org/radarbase/appserver/controller/GithubEndpoint.java
@@ -21,7 +21,8 @@
 
 package org.radarbase.appserver.controller;
 
-import org.radarbase.appserver.config.AuthConfig;
+import org.radarbase.appserver.config.AuthEntities;
+import org.radarbase.appserver.config.AuthPermission;
 import org.radarbase.appserver.service.GithubService;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
@@ -46,8 +47,8 @@ public GithubEndpoint(GithubService githubService) {
  }
 
  @Authorized(
- permission = AuthConfig.AuthPermissions.READ,
- entity = AuthConfig.AuthEntities.SUBJECT)
+ permission = AuthPermission.Constants.READ,
+ entity = AuthEntities.Constants.SUBJECT)
  @GetMapping("/" +
  PathsUtil.GITHUB_PATH
  + "/" +

diff --git a/src/main/java/org/radarbase/appserver/controller/NotificationStateEventController.java b/src/main/java/org/radarbase/appserver/controller/NotificationStateEventController.java
@@ -23,8 +23,8 @@
 
 import java.util.List;
 import javax.naming.SizeLimitExceededException;
-import org.radarbase.appserver.config.AuthConfig.AuthEntities;
-import org.radarbase.appserver.config.AuthConfig.AuthPermissions;
+import org.radarbase.appserver.config.AuthEntities;
+import org.radarbase.appserver.config.AuthPermission;
 import org.radarbase.appserver.dto.NotificationStateEventDto;
 import org.radarbase.appserver.service.NotificationStateEventService;
 import org.springframework.http.ResponseEntity;
@@ -48,7 +48,7 @@ public NotificationStateEventController(
  this.notificationStateEventService = notificationStateEventService;
  }
 
- @Authorized(permission = AuthPermissions.READ, entity = AuthEntities.PROJECT)
+ @Authorized(permission = AuthPermissions.Constants.READ, entity = AuthEntities.Constants.PROJECT)
  @GetMapping(
  value =
  "/"
@@ -64,8 +64,8 @@ public ResponseEntity<List<NotificationStateEventDto>> getNotificationStateEvent
  }
 
  @Authorized(
- permission = AuthPermissions.READ,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermissions.Constants.READ,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @GetMapping(
  value =
@@ -93,8 +93,8 @@ public ResponseEntity<List<NotificationStateEventDto>> getNotificationStateEvent
  }
 
  @Authorized(
- permission = AuthPermissions.UPDATE,
- entity = AuthEntities.SUBJECT,
+ permission = AuthPermissions.Constants.UPDATE,
+ entity = AuthEntities.Constants.SUBJECT,
  permissionOn = PermissionOn.SUBJECT)
  @PostMapping(
  value =