Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow configuring RemoteIPProxyProtocol at VHost level #2582

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

smortex
Copy link
Collaborator

@smortex smortex commented Nov 20, 2024

The module currently support configuring RemoteIP PROXY Protocol at the
system level, but the settings can also be used for specific virtual
hosts.

Allow to set RemoteIPProxyProtocol and RemoteIPProxyProtocolExceptions
at the VHost level. For cosistency, une the same parameter names and
types as the ones used for mod_remoteip configuration.

@smortex smortex requested review from bastelfreak, ekohl and a team as code owners November 20, 2024 21:44
bastelfreak
bastelfreak previously approved these changes Nov 20, 2024
The module currently support configuring RemoteIP PROXY Protocol at the
system level, but the settings can also be used for specific virtual
hosts.

Allow to set `RemoteIPProxyProtocol` and `RemoteIPProxyProtocolExceptions`
at the VHost level.  For cosistency, une the same parameter names and
types as the ones used for mod_remoteip configuration.
@smortex smortex force-pushed the allow-remote-ip-proxy-protocol-at-vhost-level branch from aaaef5b to df18222 Compare November 20, 2024 22:04
@smortex
Copy link
Collaborator Author

smortex commented Nov 20, 2024

Ooops, I fixed spacing issues in the template:

   SSLEngine on
-RemoteIPProxyProtocol On
-
+  RemoteIPProxyProtocol On
 </VirtualHost>

@smortex smortex merged commit 00d529d into main Jan 8, 2025
46 checks passed
@smortex smortex deleted the allow-remote-ip-proxy-protocol-at-vhost-level branch January 8, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants