[GitHub] [cloudstack] andrijapanicsb edited a comment on pull request #5040: systemvm: increase apache2 default timeout to 15min from 5mins

[GitBox <gi...@apache.org>]

andrijapanicsb edited a comment on pull request #5040:
URL: https://github.com/apache/cloudstack/pull/5040#issuecomment-849057179


   Hopefully, I don't sound wrong - but do we all have an understanding of the Apache Timeout and why it should NOT be increased to some high values? I understand the ACS code is LGTM, but is the change itself LGTM itself, will it cause an easier denial of service on httpd (which is wide open to public / on a public IP range for SSVM), etc?
   
   @rhtyd  I know where this change comes from, but until we verify that this indeed solves that specific user's problem (let's wait for that first...), I don't see how this makes sense. Let's discuss offline, please.
   
   From Apache site:
   ```
   
   The TimeOut directive defines the length of time Apache httpd will wait for I/O in various circumstances:
   
       When reading data from the client, the length of time to wait for a TCP packet to arrive if the read buffer is empty.
   
       For initial data on a new connection, this directive doesn't take effect until after any configured AcceptFilter has passed the new connection to the server.
       When writing data to the client, the length of time to wait for an acknowledgement of a packet if the send buffer is full.
       In mod_cgi and mod_cgid, the length of time to wait for any individual block of output from a CGI script.
       In mod_ext_filter, the length of time to wait for output from a filtering process.
       In mod_proxy, the default timeout value if ProxyTimeout is not configured.
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org