From: Brett Porter <brett@apache.org>
Date: Mon, 11 Dec 2006 06:52:00 +0000 (+0000)
Subject: note why it is how it is
X-Git-Tag: archiva-0.9-alpha-1~223
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=eb278b5812b467b4da101e1583d5b661d8eb235c;p=archiva.git

note why it is how it is

git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@485532 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/archiva-security/src/main/java/org/apache/maven/archiva/security/ArchivaSystemAdministratorRoleProfile.java b/archiva-security/src/main/java/org/apache/maven/archiva/security/ArchivaSystemAdministratorRoleProfile.java
index facb4dfad..6470f26d3 100644
--- a/archiva-security/src/main/java/org/apache/maven/archiva/security/ArchivaSystemAdministratorRoleProfile.java
+++ b/archiva-security/src/main/java/org/apache/maven/archiva/security/ArchivaSystemAdministratorRoleProfile.java
@@ -45,6 +45,8 @@ public class ArchivaSystemAdministratorRoleProfile
         operations.add( ArchivaRoleConstants.OPERATION_ADD_REPOSITORY );
         operations.add( ArchivaRoleConstants.OPERATION_EDIT_REPOSITORY );
         operations.add( ArchivaRoleConstants.OPERATION_DELETE_REPOSITORY );
+        // we don't add access/upload repository operations. This isn't a sys-admin function, and we don't want to
+        // encourage the use of the sys admin role for such operations. They can grant it as necessary.
         return operations;
     }