Bug: Updating nginx with NGINX_GEOIP=n

rdan · Mar 20, 2015

Still adding this line on nginx.conf:
include /usr/local/nginx/conf/geoip.conf;

Resulting nginx restart error.

eva2000 · Mar 20, 2015

latest .08 beta 02 adds GeoIP by default now regardless - GeoIP PHP and Nginx support is enabled out of box

related to March 13th merge commits Commits · centminmod/centminmod · GitHub

rdan · Mar 20, 2015

So every time I upgrade nginx or php that line will always be added?

eva2000 · Mar 20, 2015

yup

but should only add once

is the bug you get = multiple lines ?

update to latest .08 beta02 code base first Upgrade - Working with git command line for updating Centmin Mod local copies | Centmin Mod Community

rdan · Mar 20, 2015

Just tried to upgrade once.
So if I will just comment it out using "#" it will not be added again?
Unless I removed the whole line?

eva2000 · Mar 20, 2015

yeah as long geoip.conf exists in nginx.conf regardless if it's commented out or not, it won't be added again - code for the check centminmod/geoip.inc at 123.08centos7beta02 · centminmod/centminmod · GitHub

Log in / Register

Forums

Want to subscribe to topics you're interested in?

Bug: Updating nginx with NGINX_GEOIP=n

rdan Well-Known Member

eva2000 Administrator Staff Member

rdan Well-Known Member

eva2000 Administrator Staff Member

rdan Well-Known Member

eva2000 Administrator Staff Member

.

Log in / Register

Useful Searches

Want to subscribe to topics you're interested in?

Bug: Updating nginx with NGINX_GEOIP=n

rdan Well-Known Member

eva2000 Administrator Staff Member

rdan Well-Known Member

eva2000 Administrator Staff Member

rdan Well-Known Member

eva2000 Administrator Staff Member

.