On Mon, May 16, 2011 at 6:58 PM, Dimuthu Leelarathne <[email protected]>wrote:

> hi,
>
> On Mon, May 16, 2011 at 8:23 AM, Amila Suriarachchi <[email protected]>wrote:
>
>> hi,
>>
>> This is regarding this issue[1].
>>
>> Can we change the admin role name? if so what is the meaning of the
>> ADMIN_ROLE = "admin" in
>>  org.wso2.carbon.utils.ServerConstants?
>>
>>
> We don't have a hard-coded admin rolename.
> +1 for removing it or if it is used for some other purpose rename it.
>

I renamed the admin and every one Roles. But still see the admin role as.

Please see the attachments.

thanks,
Amila,

>
> tx,
> dimuthul
>
>
>> And also,
>>
>> userRealm.getAuthorizationManager().isUserAuthorized(
>>                     loggedInUser, topicResourcePath,
>>                     EventBrokerConstants.EB_PERMISSION_CHANGE_PERMISSION)
>>
>> returns false if user is not explicitly given the permission to that
>> resource. But in carbon there is a convention to
>> allow any user in admin role to do any activity. Then why don't we add
>> that rule too to the user manager.
>>
>> Then everyone does not have to repeat admin role check every where.
>>
>> thanks,
>> Amila.
>>
>>
>> [1] https://wso2.org/jira/browse/CARBON-9959
>>
>> _______________________________________________
>> Carbon-dev mailing list
>> [email protected]
>> http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
>>
>>
>
> _______________________________________________
> Carbon-dev mailing list
> [email protected]
> http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
>
>

<<attachment: roles.jpg>>

_______________________________________________
Carbon-dev mailing list
[email protected]
http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev

Reply via email to