José M. Guisado

Handling this case may seem reasonable. But testing with the past month database would be useless if data is outdated, wouldn't it?. I imagine that some of the cidr do...

Hi @redbeardmcg, it's come to mind to offer some kind of filtering and more features in the script but to be honest I've always ended up preferring the script to...

Hi sunshine, I think this is a good enhacement idea. I just need to take some time organizing the project before, like adding some contribution guide and dealing with formatting...

It's indeed interesting and a good opportunity to take into account use cases I did not foresee. > In fact, even if this script after modified can be run successfully,...

> Another possibility, if I may suggest one, would be to limit the map to a limited set of countries (specified on the command line) instead than by continents. This...

For the record, new changes have reduced memory usage of the script execution by 150MB more or less. ``` command time -v ./nft_geoip.py --download --file-location location.csv -o output/ ``` Yields...

Failing CI most probably is [explicit included libnftnl library](https://github.com/collectd/collectd/pull/3483/commits/5e15fbdbdd6492831483f03b0195681ef6016ed1#diff-c949f93d03f44a4217d7a138f9e2e54aR1548) **Update:** I've pushed a new commit which checks for libnftnl in configure.ac. See below.

Closing and opening because of travis

> While I have two nits, I also have the gut feeling, you're freeing more memory than reserving. It would be good to go through that again and check for...

Hope this is useful. I've run valgrind with a minimal configuration: - No writer plugin enabled - Only nftables plugin enabled `valgrind --log-file=valgrind.log --trace-children=yes --leak-check=full --show-leak-kinds=all /opt/collectd/sbin/collectd -f` ``` ......