Ed 1002
Merge request reports
Activity
Filter activity
8 9 /** 10 * 11 * @author ememisya 12 */ 13 public class VirginiaTechGroup 14 { 15 16 //MAY 17 18 19 //MUST 20 21 22 23 } 126 final Map<String, String[]> authorizationMap, 127 final AuthorizationMethod method) 128 throws LdapException, EdAuthAuthorizationException 150 final String user, 151 final Credential credential, 152 final Map<String, String[]> authorizationMap, 153 final AuthorizationMethod method) 154 throws LdapException, EdAuthAuthorizationException 129 155 { 130 156 131 157 boolean authorized = false; 132 158 133 159 if (authorizationMap.keySet().isEmpty()) { 134 160 throw new EdAuthAuthorizationException( 135 EdAuthAuthorizationException.EDAUTH_EXCEPTION_MSG_KEYSET_EMPTY); 161 EdAuthAuthorizationException.EDAUTH_EXCEPTION_MSG_KEYSET_EMPTY); Yup, I will run it through checkstyle ultimately. As a habit I always do ALT+SHIFT+F every other line which autoformats it, but checkstyle doesn't like long lines. Again, this is just a point in which I had no errors. As it stands there are still no javadocs, mapping function is not implemented, no test classes, nor required exception handling for the time being, nor the certificate options, nor the new realm support. Rest assured when I make the merge request these will be taken care of. It's being chiseled, not polished yet :)