AuthenticationComponent::logout about authentication

[burzum]

No? How does it break? AuthenticationComponent::logout() returns void|string|array. Could you please add a test case that demonstrates the issue?

    public function logout()
    {
        $controller = $this->_registry->getController();
        $result = $this->_authentication->clearIdentity(
            $controller->request,
            $controller->response
        );

        $controller->request = $result['request'];
        $controller->response = $result['response'];

        $logoutRedirect = $this->getConfig('logoutRedirect');
        if ($logoutRedirect !== false) {
            return Router::normalize($logoutRedirect);
        }
    }

[cleptric]

The Problem is the return value of SessionAuthenticator::clearIdentity().
It returns an array with the keys request and response.
Inside AuthenticationService::clearIdentity(), the whole return value is set as the response, not just the response object inside the return value.

[cleptric]

PR is open.