Changeset 208

Timestamp:

03/03/10 13:19:15 (14 years ago)

Author:

patdui

Message:

• some better error handling

Location:

ComponentRegistry/trunk/ComponentRegistry/src/main/java/clarin/cmdi/componentregistry

Files:

• ComponentRegistryImpl.java (modified) (1 diff)
• rest/ComponentRegistryRestService.java (modified) (2 diffs)

ComponentRegistry/trunk/ComponentRegistry/src/main/java/clarin/cmdi/componentregistry/ComponentRegistryImpl.java

@@ -330 +330 @@
     private void checkAuthorisation(AbstractDescription desc, Principal principal) throws UserUnauthorizedException {
         if (!principal.getName().equals(desc.getCreatorName())) {
-            throw new UserUnauthorizedException("Unauthorized operation user: " + principal.getName()
-                    + " was not the creator of profile/component (description=" + desc + ").");
+            throw new UserUnauthorizedException("Unauthorized operation user '" + principal.getName()
+                    + "' was not the creator of profile/component (description=" + desc + ").");
         }
     }

ComponentRegistry/trunk/ComponentRegistry/src/main/java/clarin/cmdi/componentregistry/rest/ComponentRegistryRestService.java

@@ -99 +99 @@
             return Response.serverError().status(Status.INTERNAL_SERVER_ERROR).build();
         } catch (UserUnauthorizedException e) {
-            LOG.info("Component with id: " + componentId + " deletion failed.", e);
+            LOG.info("Component with id: " + componentId + " deletion failed: "+e.getMessage());
             return Response.serverError().status(Status.UNAUTHORIZED).build();
         }
@@ -145 +145 @@
             return Response.serverError().status(Status.INTERNAL_SERVER_ERROR).build();
         } catch (UserUnauthorizedException e) {
-            LOG.info("Profile with id: " + profileId + " deletion failed.", e);
+            LOG.info("Profile with id: " + profileId + " deletion failed: "+e.getMessage());
             return Response.serverError().status(Status.UNAUTHORIZED).build();
         }