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

Create htaccess.conf #1177

Closed
wants to merge 4 commits into from
Closed

Create htaccess.conf #1177

wants to merge 4 commits into from

Conversation

denics
Copy link
Contributor

@denics denics commented Sep 2, 2015

In our configuration I cannot use IPTables because all of our traffic come from a firewall that I do not control, with the firewall IP. This solution is not the most elegant, as htaccess slow down the server, but it is better than having attackers...

In our configuration I cannot use IPTables because all of our traffic come from a firewall that I do not control, with the firewall IP. This solution is not the most elegant, as htaccess slow down the server, but it is better than having attackers...
#
# Author: Denis Pitzalis
#
# $Revision$
Copy link
Member

Choose a reason for hiding this comment

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

no need for those ... we should strip them all eventually -- remnant from SVN

made changes requested for pull
# Fail2Ban configuration file
# htaccess
# You can ban IPs using the Deny directive of Apache in your htaccess.
# You should add, at the very end of the .htaccess "Order Deny,Allow"
Copy link
Member

Choose a reason for hiding this comment

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

so shouldn't there be

actionstart = grep "^Order " <htaccess1> || echo "Order Deny,Allow" >> <htaccess1>

@yarikoptic yarikoptic mentioned this pull request Sep 4, 2015
@yarikoptic
Copy link
Member

so @denics -- interested to finish up ? there should be only a single htaccess file entry and there is an outstanding comment to address

@denics
Copy link
Contributor Author

denics commented Sep 14, 2015

yep, sorry! I was travelling. I get back to this asap.

@sebres sebres mentioned this pull request Oct 26, 2015
@yarikoptic
Copy link
Member

@denics still interested to pursue this PR?

@denics
Copy link
Contributor Author

denics commented Jan 11, 2016

Hi @yarikoptic sure, sorry. In the meantime, the system has been running for the last 4 months on our servers and it works as good as expected.

I commented out the extra servers, and added me to the "thanks" file, what are the standing issues?

Added myself
@sebres
Copy link
Contributor

sebres commented Jun 20, 2017

see #1811

@sebres sebres closed this Jun 20, 2017
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.

None yet

3 participants