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

Block Ip Middleware #51

Closed
rbtsolis opened this issue Mar 21, 2018 · 3 comments
Closed

Block Ip Middleware #51

rbtsolis opened this issue Mar 21, 2018 · 3 comments

Comments

@rbtsolis
Copy link

Hi, currently im working on https://github.com/philipn/django-block-ip

But i think, it's a good idea add this function in this library, what do you think about that?

@EvaSDK
Copy link
Contributor

EvaSDK commented May 15, 2018

You should probably look into django-axes as well.

@9mido
Copy link

9mido commented May 18, 2020

Not a bad idea to be able to block the IPs as long as collecting the IPs in the database is encrypted and does not violate PII / GDPR laws. But you might be able to get away with not encrypting if your linux server itself is secure.

Also, you need to be able to get the real IP address of the client even if the server is behind cloudflare or a load balancer. You don't want to block people if they are behind the same cloudflare IP address then everyone on your site will be blocked/rate limited which is bad. There is also the issue of IP spoofing where a malicious user could fake their IP address,

@dmpayton
Copy link
Owner

I think IP blocking falls outside the scope of django-admin-honeypot. There are many ways to handle IP blocking, in terms of both block list management and responses to blocked IPs; we're mainly concerned with raising red flags when someone might be attempting to get into the Django admin.

But it'd be cool to see a reference implementation that integrates, e.g., the honeypot signal into an existing ip block app, to automate the blocking of IPs that are honeypotted so many times in a time period. I don't know that I'd want to bake it in to django-admin-honeypot, but it would be neat to check out.

@9mido I'm not as up to speed on GDPR, but I know there's already an issue for it, so we can continue the conversation there.

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

No branches or pull requests

4 participants