Skip to content

Conversation

@jonasbardino
Copy link
Contributor

Integrate apache mod reqtimeout use if requested to enable various default request timeouts in order to help foil slowloris-style attacks where all workers and resources are gradually depleted by attacker or busy-idling clients of other sorts. Matches modern Debian/Ubuntu and RHEL/Rocky Apache installation defaults, but requires explicit enabling until further tested.

…fault

request timeouts in order to help foil slowloris-style attacks where all
workers and resources are gradually depleted by attacker or busy-idling
clients of other sorts. Matches modern Debian/Ubuntu and RHEL/Rocky Apache
installation defaults, but requires explicit enabling until further tested.
@jonasbardino jonasbardino self-assigned this Nov 10, 2025
@jonasbardino jonasbardino added the enhancement New feature or request label Nov 10, 2025
@jonasbardino jonasbardino marked this pull request as ready for review November 10, 2025 13:53
@jonasbardino jonasbardino requested a review from a team November 10, 2025 13:54
enable_jupyter=False,
enable_cloud=False,
enable_hsts=True,
enable_reqtimeout=False,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should consider naming this 'enable_http_reqtimeout' or 'enable_apache_reqtimeout' to keep the namespace open for other types of 'reqtimeout' options ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for reviewing.
I did in fact start out with an apache_ prefix here and on the mod_evasive PR #389 but dropped it again looking at how enable_wsgi, enable_hsts, enable_openid, etc. are very similar for other Apache modules. I agree that it is perhaps ambiguous and to general here - although one could cheekily say the same about enable_quota ;-)

Perhaps we should again call this what it does rather than what it uses for the purpose? E.g. enable_slowloris_guard.

The same decision will be relevant for the QoS module addition we talked about off-list, btw.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants