https://bz.apache.org/bugzilla/show_bug.cgi?id=55477

Christopher Schultz <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable

--- Comment #11 from Christopher Schultz <[email protected]> ---
Given Context.addRoleMapping, it looks like MappingRealm just becomes a place
for role mappings to be collected and set on the Context when the component is
initialized. The code can be simplified greatly in this case.

While I think this implementation will work, I think it might be better-suited
to add it to RealmBase to allow any realm to re-map roles. It will also make
configuration slightly simpler because you only need one <Realm> instead of
two.

Stefan, are you still willing to pursue this patch?

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to