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

IP whitelist for ranges doesn't works #44

Open
momkin opened this issue May 16, 2020 · 7 comments
Open

IP whitelist for ranges doesn't works #44

momkin opened this issue May 16, 2020 · 7 comments

Comments

@momkin
Copy link

momkin commented May 16, 2020

Issue title

The IP ranges i allowed bellow doesn't get whitelisted by the script !
I've tried to many times but still not working !

Issue Description

Versions:

  • Browser(s):
  • Nginx version:
  • Operating system of web server running Nginx:

Nginx config:

local ip_whitelist = {
"66.102.0.0/20","66.102.0.0/24","66.102.1.0/24","66.102.2.0/24","66.102.3.0/24","66.102.4.0/24","66.102.8.0/23","66.102.12.0/24","66.249.64.0/19","66.249.64.0/20","66.249.80.0/22","66.249.84.0/23","66.249.88.0/24",
}

Screenshot(s):

[Screenshot(s) for difficult to describe visual issues are mandatory. Post links instead of Inline Images for Screenshots containing Adult material.]

Settings:

  • [List here all the changes you made to the default settings]

Other optional information you want to add other than the above:

@Sylvvvia
Copy link

Ip blacklist/whitelist works fine, after you have added ips into the table you need to restart webserver (no reload, but restart).

@AndreiG6
Copy link

Why would you restart vs reload?

@Sylvvvia
Copy link

Reload doesn't reload script it reloads only nginx config / openresty config. In my openresty reload doesn't work for script but restart re-reads it and blacklist/whitelist works as expected.

@AndreiG6
Copy link

Sounds like something is being done wrong if that's what you're getting IMO. A reload should do just that, gracefully reload the config openresty/nginx config, re-init_by_lua, etc

@momkin
Copy link
Author

momkin commented May 20, 2020

The

Ip blacklist/whitelist works fine, after you have added ips into the table you need to restart webserver (no reload, but restart).

If you you put to many ranges like 15 ranges of IPS it doesn't works !
If i put less ranges like 5 it works !

@Sylvvvia
Copy link

Actually i have 43 ranges whitelisted to be exact :-)

@glushchenko
Copy link

I fixed it here bis-gmbh@80f23fd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants