lua-geoip icon indicating copy to clipboard operation
lua-geoip copied to clipboard

Commit of working asnum.c, to perform ASN lookups

Open robison opened this issue 8 years ago • 4 comments

This has been tested empirically and found to work for name, addr, and ipnum lookups. Still working on the actual test cases; would appreciate any feedback.

robison avatar Dec 15 '15 19:12 robison

Thank you for your contribution! Please provide basic autotests and add yourself to the AUTHORS file so I can merge this.

On 15 дек. 2015 г., at 22:09, Robison Jacka [email protected] wrote:

This has been tested empirically and found to work for name, addr, and ipnum lookups. Still working on the actual test cases; would appreciate any feedback.

You can view, comment on, or merge this pull request online at:

https://github.com/agladysh/lua-geoip/pull/13

Commit Summary

Commit of working asnum.c, to perform ASN lookups File Changes

A src/asnum.c (228) Patch Links:

https://github.com/agladysh/lua-geoip/pull/13.patch https://github.com/agladysh/lua-geoip/pull/13.diff — Reply to this email directly or view it on GitHub.

agladysh avatar Dec 15 '15 22:12 agladysh

This should be finished now. Found a dumb memory leak on my part, and finally squashed it. Also finished the tests & profiling for geodb.asnum. Please take a look, and let me know if you see anything that needs updating/fixing. Thanks!

robison avatar Jan 13 '16 06:01 robison

Thank you for your pull request. Please see my questions.

agladysh avatar Jan 20 '16 14:01 agladysh

@robison ping?

msva avatar Jul 22 '16 18:07 msva