[ 
https://issues.apache.org/jira/browse/MESOS-5336?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15280485#comment-15280485
 ] 

Zhitao Li commented on MESOS-5336:
----------------------------------

I think I can come up with an implementation for {{GET_QUOTA_WITH_ROLE}} using 
{{stout::collect}} on a list of futures.

Question: if we have {{GET_QUOTA_WITH_ROLE}}, do you think we still want to 
guard {{/quota}} endpoint with {{GET_ENDPOINT_WITH_PATH}}? The closest 
alternative would be an ACL of {{ANY}} or {{NONE}} role, but it probably would 
return empty map rather than {{Forbidden}}.

I have no strong opinion here. I'll try a diff on top my previous review while 
wait for your answer.

> Add authorization to GET /quota
> -------------------------------
>
>                 Key: MESOS-5336
>                 URL: https://issues.apache.org/jira/browse/MESOS-5336
>             Project: Mesos
>          Issue Type: Improvement
>          Components: master, security
>            Reporter: Adam B
>              Labels: mesosphere, security
>             Fix For: 0.29.0
>
>
> We already authorize which http users can set/remove quota for particular 
> roles, but even knowing of the existence of these roles (let alone their 
> quotas) may be sensitive information. We should add authz around GET 
> operations on /quota.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to