You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by "Jarek Gawor (JIRA)" <ji...@apache.org> on 2008/12/03 16:33:44 UTC
[jira] Resolved: (GERONIMO-4441) The assignment to variable
"handler" has no effect in ConstantLoginModule.java
[ https://issues.apache.org/jira/browse/GERONIMO-4441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jarek Gawor resolved GERONIMO-4441.
-----------------------------------
Resolution: Fixed
Fix Version/s: 2.2
2.1.4
Assignee: Jarek Gawor
Patch committed to trunk (revision 722931) and branches/2.1 (revision 722932). Thanks!
> 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
> Assignee: Jarek Gawor
> Fix For: 2.1.4, 2.2
>
> 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.