[ https://issues.apache.org/jira/browse/GERONIMO-4441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Shawn Jiang updated GERONIMO-4441: ---------------------------------- Attachment: GERONIMO-4441.patch patch for trunk and 2.1 branch with unit testing passed. > The assignment to variable "handler" has no effect in ConstantLoginModule.java > ------------------------------------------------------------------------------ > > Key: GERONIMO-4441 > URL: https://issues.apache.org/jira/browse/GERONIMO-4441 > Project: Geronimo > Issue Type: Bug > Security Level: public(Regular issues) > Affects Versions: 2.1.3, 2.1.4, 2.2 > Reporter: Shawn Jiang > Attachments: GERONIMO-4441.patch > > > Seems it's a typo bug in > https://svn.apache.org/repos/asf/geronimo/server/trunk/framework/modules/geronimo-security/src/main/java/org/apache/geronimo/security/realm/providers/ConstantLoginModule.java > /** > * Save the userName and groupNames settings for use during commit() > */ > public void initialize(Subject subject, CallbackHandler callbackHandler, > Map sharedState, Map options) { > this.subject = subject; > this.handler = handler; > > this.userName = (String)options.get(USER_NAME); > this.groupNames = (String)options.get(GROUP_NAMES); > } > " this.handler = handler;" should be " this.handler = callbackHandler;" -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.