[jira] [Commented] (MESOS-7914) Replace usage of `ObjectApprover` with `AuthorizationAcceptor`

["Greg Mann (JIRA)" <ji...@apache.org>]

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

Greg Mann commented on MESOS-7914:
----------------------------------

Note that it would also be useful to add some error logging [here|https://github.com/apache/mesos/blob/5125b80ea50b5babd7636234605b66c627780834/src/common/http.cpp#L1212-L1216] in an {{onFailed}} handler to log the case where the authorizer fails to return a valid object approver.

> Replace usage of `ObjectApprover` with `AuthorizationAcceptor`
> --------------------------------------------------------------
>
>                 Key: MESOS-7914
>                 URL: https://issues.apache.org/jira/browse/MESOS-7914
>             Project: Mesos
>          Issue Type: Improvement
>          Components: security
>    Affects Versions: 1.4.0
>            Reporter: Greg Mann
>            Assignee: Greg Mann
>              Labels: authorization, mesosphere
>
> Now that the {{AuthorizationAcceptor}} class has been added, we can replace all occurrences of {{getObjectApprover}} with {{AuthorizationAcceptor::create}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)