https://bz.apache.org/bugzilla/show_bug.cgi?id=55477
Christopher Schultz <ch...@christopherschultz.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |PatchAvailable
--- Comment #11 from Christopher Schultz <ch...@christopherschultz.net> ---
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: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
