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

Replace NGINX with OpenResty #276

Merged
merged 8 commits into from
Feb 9, 2024
Merged

Replace NGINX with OpenResty #276

merged 8 commits into from
Feb 9, 2024

Conversation

atodorov
Copy link
Member

because of it's built-in support for Lua and generally newer code-base

https://www.nginx.com/resources/wiki/modules/lua/
https://openresty.org/en/linux-packages.html#rhel

@atodorov atodorov force-pushed the use_openresty branch 13 times, most recently from 4c58d51 to f8cfc40 Compare January 29, 2024 21:50
@atodorov atodorov force-pushed the use_openresty branch 4 times, most recently from 7b5d9fb to b6f5566 Compare February 6, 2024 19:27
@atodorov atodorov force-pushed the use_openresty branch 6 times, most recently from 7b7957e to 4d362e7 Compare February 8, 2024 19:22
because tests are trying to hit robots.txt
and include more temporaty files as part of uploaded CI artifacts

WARNING: don't change the Referer value when sending
POST /accounts/login/ via curl.

Django's login backend/form validation actually checks for this and will
return a 404 if you try to login using a CSRF token without the correct
Referer header
for 4xx, 5xx, /static/, /uploads/ and authenticated requests
and update asserts in test script to make sure these values are actually
taken into account
@atodorov atodorov marked this pull request as ready for review February 9, 2024 10:23
@atodorov atodorov merged commit 45297b7 into master Feb 9, 2024
8 of 9 checks passed
@atodorov atodorov deleted the use_openresty branch February 9, 2024 10:23
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.

1 participant