On 10/18/11 9:31 AM, Anil Saldhana wrote:
On 10/18/2011 09:18 AM, Jason T. Greene wrote:
> On 10/18/11 9:09 AM, Anil Saldhana wrote:
>> Marcus,
>> this is in regard to your proposed changes to JBossWebRealm for the
>> authorization bits.
>>
>>
https://github.com/mmoyses/jboss-as/commit/ba3c43f8dfc9c201098392c5ebf904...
>>
>>
>> Previously, AS5/6, we had the JBoss Authorization enabled by default.
>> IMO for AS7, you have taken the right approach to allow user to
>> configure whether to use JBoss Authz via jboss-web.xml setting.
>>
>> We need to get this merged asap such that I can finish the auditing task
>> I am currently working on.
>
> Just work off his commit then. That's the whole point of GIT, that a
> developer doesn't race getting their commits in.
>
I am go to cherry pick this commit. But I would like Marcus's changes
merged so that the sec propagation issue is tested.
It will be looked at the next time the queue is merged, just like all
the other patches in queue.
--
Jason T. Greene
JBoss AS Lead / EAP Platform Architect
JBoss, a division of Red Hat