Skip to content

[18.0][ADD] base_trusted_proxies#3599

Draft
qgroulard wants to merge 1 commit intoOCA:18.0from
acsone:18.0-base_trusted_proxies-qgr
Draft

[18.0][ADD] base_trusted_proxies#3599
qgroulard wants to merge 1 commit intoOCA:18.0from
acsone:18.0-base_trusted_proxies-qgr

Conversation

@qgroulard
Copy link
Copy Markdown

Allow configuration of the number of trusted proxies for accurate Client IP detection.

Fixes #3540

@OCA-git-bot OCA-git-bot added series:18.0 mod:base_trusted_proxies Module base_trusted_proxies labels Apr 15, 2026
@qgroulard qgroulard marked this pull request as draft April 15, 2026 16:04
Copy link
Copy Markdown
Member

@sbidoul sbidoul left a comment

Choose a reason for hiding this comment

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

LGTM. A couple of comments.

Comment thread base_trusted_proxies/post_load.py Outdated
Comment thread base_trusted_proxies/post_load.py Outdated
@qgroulard qgroulard force-pushed the 18.0-base_trusted_proxies-qgr branch from 3395381 to dd3aa3b Compare April 17, 2026 10:23
@qgroulard
Copy link
Copy Markdown
Author

LGTM. A couple of comments.

Thanks for the review, I have added all those checks.

@qgroulard qgroulard force-pushed the 18.0-base_trusted_proxies-qgr branch 2 times, most recently from de5fc55 to 08277ff Compare April 17, 2026 10:38
Allow configuration of the number of trusted proxies for accurate Client IP detection.
@qgroulard qgroulard force-pushed the 18.0-base_trusted_proxies-qgr branch from 08277ff to f31281d Compare April 17, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:base_trusted_proxies Module base_trusted_proxies series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New module to configure the number of trusted proxies in x-forwarded-for

3 participants