Overriding HttpServer::MAXIMUM_BUFFER_SIZE #436

Activity

1. ozahorulia commented on Jan 11, 2022

   

   ozahorulia

   on Jan 11, 2022

   Last edited by ozahorulia

   Author

   More actions

   Fixed by explicitly passing middleware objects to HttpServer:

           $requestMaxSize = $this->iniSizeToBytes($this->requestMaxSize);
           $memoryLimit = $this->iniSizeToBytes($this->memoryLimit);
   
           $requestHandlers = [];
           $requestHandlers[] = new StreamingRequestMiddleware();
   
           $concurrency = $this->getConcurrentRequestsLimit($memoryLimit, $requestMaxSize);
           if ($concurrency !== null) {
               $requestHandlers[] = new LimitConcurrentRequestsMiddleware($concurrency);
           }
   
           $requestHandlers[] = new RequestBodyBufferMiddleware($requestMaxSize);
           if ($this->enablePostDataReading) {
               $requestHandlers[] = new RequestBodyParserMiddleware();
           }
   
           $requestHandlers[] = $callback;
   
           return new React\Http\HttpServer(...$requestHandlers);
   
2. ozahorulia

   closed this as completedon Jan 11, 2022
3. SimonFrings commented on Jan 31, 2022

   

   SimonFrings

   on Jan 31, 2022

   Member

   More actions

   Hey @hast, thanks for opening this up 👍

   It seems like you already found your way of dealing with this, a similar approach can be found inside the documentation. You can also take a look at #371.

   👍React with 👍1Reacted by Oleksandr Zahorulia
4. clue

   added

   question

   on Jan 31, 2022