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

IP Based ACL #38

Closed
vrelk opened this issue Dec 23, 2018 · 6 comments
Closed

IP Based ACL #38

vrelk opened this issue Dec 23, 2018 · 6 comments
Labels
enhancement

Comments

@vrelk
Copy link

vrelk commented Dec 23, 2018

Please add support for IP based ACLs and Cloudflare IP correction.

https://support.cloudflare.com/hc/en-us/articles/200170706-How-do-I-restore-original-visitor-IP-with-Nginx-
@fidelix
Copy link

fidelix commented Dec 24, 2018

This would be pretty cool. I'm currently doing this by adding an include manually to every host, but having this built-in is probably a natural feature.

@sjabby
Copy link

sjabby commented Jan 3, 2019

yes please.

@jc21
Copy link
Member

jc21 commented Jan 3, 2019

Right so yeah I was also wanting IP range based options for the Access Lists but I forgot to consider the cloudfare / cloudfront ip forwarding as well. I'm able to get the latest cloudfront ip ranges with a script that I can program to fire once a day and I just saw cloudfare has a list too

@jc21 jc21 mentioned this issue Jan 8, 2019
Open
@danmed danmed mentioned this issue Nov 20, 2019
Closed
@Subv
Copy link
Contributor

Subv commented May 30, 2020

Can this be closed now that #360 is merged?

@jc21
Copy link
Member

jc21 commented May 31, 2020

Sounds good to me

@jc21 jc21 closed this as completed May 31, 2020
@risiman
Copy link

risiman commented Jan 31, 2021

That sounds great to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@fidelix @Subv @vrelk @jc21 @sjabby @risiman