TYPO3.Flow

The proxies for the Http\Request are generated but not used. So you are not able to use AOP for this class. For example try this:

    /**
     * @param \TYPO3\FLOW3\Aop\JoinPointInterface $joinPoint The current joinpoint
     *
     * @FLOW3\Around("method(TYPO3\FLOW3\Http\Request->decodeBodyArguments())")
     * @return mixed Result of the advice chain
     */
    public function transformRequestBody(\TYPO3\FLOW3\Aop\JoinPointInterface $joinPoint) {
        /** code **/
    }

The advices are never called.

The bug is located in the Http\RequestHandler. There the Request is generated through a static constructor method.

Maybe the request should be instantiated by the object manager while resolving the dependencies like the other properties of the request handler.

Like this:

    protected function resolveDependencies() {
        ...
        $this->request = $objectManager->get('TYPO3\FLOW3\Http\Request');
    }

Then you could feed the constructor of the request with a kind of "RequestVariablesLocator" which encapsulates the superglobals. When it's a singleton it get's injected.

Like this:

    public function __construct(\TYPO3\FLOW3\Mvc\RequestVariablesLocator $requestVariableLocator) {
        $server = $requestVariableLocator->getServerVariables();

        $this->headers = Headers::createFromServer($server);
        $this->setMethod(isset($server['REQUEST_METHOD']) ? $server['REQUEST_METHOD'] : 'GET');
        $protocol = (isset($server['SSL_SESSION_ID']) || (isset($server['HTTPS']) && ($server['HTTPS'] === 'on' || strcmp($server['HTTPS'], '1') === 0))) ? 'https' : 'http';
        $this->uri = new Uri($protocol . '://' . (isset($server['HTTP_HOST']) ? $server['HTTP_HOST'] : 'localhost') . str_replace('/index.php', '', (isset($server['REQUEST_URI']) ? $server['REQUEST_URI'] : '/')));
        $this->server = $server;
        $this->arguments = $this->buildUnifiedArguments($requestVariableLocator->getGetVariables(), $requestVariableLocator->getPostVariables(), $requestVariableLocator->getPostVariables());
    }