diff options
| author | James Moger <james.moger@gitblit.com> | 2012-10-10 17:13:18 -0400 |
|---|---|---|
| committer | James Moger <james.moger@gitblit.com> | 2012-10-10 17:13:18 -0400 |
| commit | 58e7ec808177e53981792b1489695a51d2c91fe5 (patch) | |
| tree | ac0a052a97ea5c0fc1760f8ee21654e345098a44 /src/com/gitblit/models | |
| parent | 17363c3957dab02bf8a5ab3a89dcace147c4f20e (diff) | |
| download | gitblit-58e7ec808177e53981792b1489695a51d2c91fe5.tar.gz gitblit-58e7ec808177e53981792b1489695a51d2c91fe5.zip | |
Add regex permission matching to hasRepositoryPermission check (issue 36)
Diffstat (limited to 'src/com/gitblit/models')
| -rw-r--r-- | src/com/gitblit/models/TeamModel.java | 16 | ||||
| -rw-r--r-- | src/com/gitblit/models/UserModel.java | 16 |
2 files changed, 30 insertions, 2 deletions
diff --git a/src/com/gitblit/models/TeamModel.java b/src/com/gitblit/models/TeamModel.java index 0b9c5069..896adfe6 100644 --- a/src/com/gitblit/models/TeamModel.java +++ b/src/com/gitblit/models/TeamModel.java @@ -92,7 +92,21 @@ public class TeamModel implements Serializable, Comparable<TeamModel> { */
public boolean hasRepositoryPermission(String name) {
String repository = AccessPermission.repositoryFromRole(name).toLowerCase();
- return permissions.containsKey(repository) || repositories.contains(repository);
+ if (permissions.containsKey(repository)) {
+ // exact repository permission specified
+ return true;
+ } else {
+ // search for regex permission match
+ for (String key : permissions.keySet()) {
+ if (name.matches(key)) {
+ AccessPermission p = permissions.get(key);
+ if (p != null) {
+ return true;
+ }
+ }
+ }
+ }
+ return false;
}
/**
diff --git a/src/com/gitblit/models/UserModel.java b/src/com/gitblit/models/UserModel.java index a4a40245..d8c2abe3 100644 --- a/src/com/gitblit/models/UserModel.java +++ b/src/com/gitblit/models/UserModel.java @@ -132,7 +132,21 @@ public class UserModel implements Principal, Serializable, Comparable<UserModel> */
public boolean hasRepositoryPermission(String name) {
String repository = AccessPermission.repositoryFromRole(name).toLowerCase();
- return permissions.containsKey(repository) || repositories.contains(repository);
+ if (permissions.containsKey(repository)) {
+ // exact repository permission specified
+ return true;
+ } else {
+ // search for regex permission match
+ for (String key : permissions.keySet()) {
+ if (name.matches(key)) {
+ AccessPermission p = permissions.get(key);
+ if (p != null) {
+ return true;
+ }
+ }
+ }
+ }
+ return false;
}
/**
|