sipcapture / homer5-docker

HOMER 5 Docker Containers (OBSOLETE)
http://sipcapture.org
40 stars 61 forks source link

Cleaning up apt packages. #57

Closed sudermanjr closed 6 years ago

sudermanjr commented 6 years ago

According to https://github.com/sipcapture/homer/wiki/Example:-GeoIP the commented out section is unnecessary in Debian.

I added a the geo-ip extras module which gives you the GeoIPCity.dat which is already referenced in the kamailio.cfg.

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

sudermanjr commented 6 years ago

Not sure why this build is failing. I looked and it appears to have nothing to do with these changes


The command "docker exec -it mysql mysql -u root -p'secret' -s -e "SELECT * FROM homer_data.sip_capture_call_$(date '+%Y%m%d')\G" | grep -i "proxy auth"" exited with 1.
0.02s$ curl -s -o /dev/null -w "%{http_code}" localhost | grep -iP "^200$"
200```
sudermanjr commented 6 years ago

Any chance of getting this merged?

dougbtv commented 6 years ago

Thanks for circling back @sudermanjr -- looks like the failing tests are actually unrelated. Let's merge it in, looks like a net benefit. Thanks for the submission, too!

sudermanjr commented 6 years ago

Thanks!