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.

Sign up for GitHub

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

Jump to bottom

Add limit req to nginx conf to prevent DDOS #112

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add limit req to nginx conf to prevent DDOS #112

wants to merge 1 commit into from

Conversation

atxr
Copy link
Contributor

@atxr atxr commented Jun 5, 2023

In order to prevent DDOS attacks, I added in this PR a limit_req to the nginx production template.

@atxr
Update nginx.conf.prod.template
01c9f45 
Add limit req to prevent DDOS
@atxr atxr requested review from ctmbl and ZynoXelek June 5, 2023 20:47
@atxr atxr self-assigned this Jun 5, 2023
@atxr atxr added Security issue A potential security issue Priority: High The Issue must be addressed as soon as possible Severity: Critical The bug or Issue prevent the website from running or present a critical security issue labels Jun 5, 2023
ctmbl
ctmbl reviewed Jun 9, 2023
View reviewed changes
Copy link
Contributor

@ctmbl ctmbl left a comment

Choose a reason for hiding this comment

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

Looks good but we need to really test it before approving!

nginx/nginx.conf.prod.template Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ctmbl ctmbl ctmbl left review comments

@ZynoXelek ZynoXelek ZynoXelek approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

@atxr atxr

Labels
Priority: High The Issue must be addressed as soon as possible Security issue A potential security issue Severity: Critical The bug or Issue prevent the website from running or present a critical security issue
Projects
iScsc website
Status: Backlog
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@atxr @ctmbl @ZynoXelek