mesos-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zhitao Li (JIRA)" <>
Subject [jira] [Commented] (MESOS-5336) Add authorization to GET /quota
Date Wed, 11 May 2016 17:31:12 GMT


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

> Add authorization to GET /quota
> -------------------------------
>                 Key: MESOS-5336
>                 URL:
>             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

View raw message