hazza/DSpace
1
0
Fork 0
mirror of https://github.com/DSpace/DSpace.git synced 2025-10-08 10:34:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

re: #736, fix logic error in handling multiple values in SHIB-SCOPED-AFFILIATION header, also make minor revisions to help messages to correct for suggested syntax and config file location

Browse Source
This commit is contained in:
Hardy Pottinger
2013-05-14 16:59:55 -05:00
parent 64e16567d6
commit e2d38cbcd8
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
dspace-api/src/main/java/org/dspace/authenticate/ShibAuthentication.java
View File

@@ -336,7 +336,7 @@ public class ShibAuthentication implements AuthenticationMethod
}
if (groupNames == null) {
log.debug("Unable to find role mapping for the value, '"+affiliation+"', there should be a mapping in the dspace.cfg: authentication.shib.role."+affiliation+" = <some group name>");
log.debug("Unable to find role mapping for the value, '"+affiliation+"', there should be a mapping in config/modules/authentication-shibboleth.cfg: role."+affiliation+" = <some group name>");
continue;
} else {
log.debug("Mapping role affiliation to DSpace group: '"+groupNames+"'");
@@ -1158,7 +1158,7 @@ public class ShibAuthentication implements AuthenticationMethod
// prevent an endless loop in an error condition.
values = values.substring(1,values.length());
} else if (idx > 0 && values.charAt(idx-1) != '\\' ) {
} else if (idx > 0 && values.charAt(idx-1) == '\\' ) {
// The attribute starts with an escaped semicolon
idx++;
} else if ( idx > 0) {