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

Axes behind reverse proxy #1521

Merged
merged 4 commits into from
Dec 15, 2023
Merged

Axes behind reverse proxy #1521

merged 4 commits into from
Dec 15, 2023

Conversation

bbkz
Copy link
Contributor

@bbkz bbkz commented Dec 9, 2023

Proposed Changes

  • Additional Axes setting for docker, to set the AXES_LOCKOUT_PARAMETERS
  • Added django-ipware version 6.0.2 a wrapper for python-ipware
  • Added Axes setting for proxy setups to docker settings

More details

Adding django-ipware makes it possible to use Axes behind reverse proxies, which is quite common in a kubernetes setup.

django-ipware was deprecated but revived since version 6.0 as a wrapper to python-ipware.

bbkz added a commit to wger-project/helm-charts that referenced this pull request Dec 15, 2023
* axes behing reverse proxy works now, but needs [pull request #1521](wger-project/wger#1521) merged
* gunicorn: uses gthread with 4 workers and 3 threads as a default
* nginx: logging shows X_FORWARDED_FOR
* nginx: proxy tuning
* nginx: set timezone
* add example values.yaml for a productive environment
@rolandgeider
Copy link
Member

great! I actually ran into this with the compose file

@rolandgeider rolandgeider merged commit efa1d7a into wger-project:master Dec 15, 2023
7 checks passed
@bbkz
Copy link
Contributor Author

bbkz commented Dec 15, 2023

Cool thank you! The helm chart wger-0.1.10-rc.11 has it implemented, i will release it soon...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants