Hi Arturo! Thanks for the feedback, On 3/1/20 10:46, Arturo Borrero Gonzalez wrote: > On 12/30/19 10:19 PM, Jose M. Guisado Gomez wrote: >> This patch adds a python script which generates .nft files that >> contains mappings between the IP address and its geolocation. >> > > Thanks for the patch! > > I have some concerns about including this in the main nftables repository tree. > My experience dealing with them in other repos (iptables, ebtables, etc) is that > they tend to go ignored from the maintenance point of view, being a burden at > the end.> Please don't get me wrong, I find this script very useful for many use cases. > I'm just wondering about the right place for storage. Perhaps a more convenient > approach could be to have an additional repo somewhere to hold this kind of > contributions. I understand, those concerns seem reasonable. If a place is to be decided to have the script I wouldn't mind having the script in a repository on Github or some other platform in the mean time. And build on feedback as soon as there is some. PS. Main purpose of the script is to provide some starting geoip solution to nftables *which does not depend on MaxMind's geolite* (there's been a policy update requiring signing up to download and accepting a new EULA, breaking xt_geoip_dl for example).
