[MESOS-5730] Sandbox access authorization should fail for non existing sandboxes.

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.0.0
• Component/s: security
• Labels: authorization mesosphere security
• Sprint: Mesosphere Sprint 38
• Assignee: Unassigned
• Reporter: Till Toenshoff
• Shepherd: Till Toenshoff
• Created: 28/Jun/16 02:15
• Updated: 21/Jul/16 19:10

Description

The local authorizer currently tries to authorize ACCESS_SANDBOX even if no further object specification - e.g. framework_info or executor_info) where specified / available at that time.

Given that there is likely no sandbox available if there was no executor_info provided, I think we should actually fail instead of allow or deny (403).

A failure would result into an IMHO more appropriate ServiceUnavailable (503).

See https://github.com/apache/mesos/commit/c8d67590064e35566274116cede9c6a733187b48#diff-dd692b1640b2628014feca01a94ba1e1R241