Add geoip module support

ANXS / nginx

A fairly flexible and feature full Ansible role for the NGINX web server.

MIT License

151 stars 82 forks source link

Closed welcoMattic closed 5 years ago

welcoMattic commented 9 years ago

this PR fix the OpenSSL version upgrade (https://github.com/welcoMattic/nginx/commit/adce707ee524289591dc78affbcaedcce72b63a9), required to pass the build step, because previous version of OpenSSL is not online anymore.

welcoMattic commented 9 years ago

I don't understand why Travis build fails. There's no failed task

welcoMattic commented 9 years ago

@pjan up

otakup0pe commented 9 years ago

The first travis pass succeeds. The idempotency test fails. After fixing the vagrant-inventory locally I see two changes on the second vagrant run.

welcoMattic commented 9 years ago

I just fix the last version number of OpenSSL here : https://github.com/welcoMattic/nginx/pull/1. I pull it into my geoip to avoid build failing.

welcoMattic commented 9 years ago

@otakup0pe what's your testing workflow on local? I can't reproduce TravisCI workflow with success.

welcoMattic commented 9 years ago

@otakup0pe @pjan LGTM

otakup0pe commented 7 years ago

Did we merge this in a different PR? If not, let's get the conflicts sorted so we can merge.

welcoMattic commented 7 years ago

It was merged in https://github.com/ANXS/nginx/pull/66. We can close this PR, I think