geoip icon indicating copy to clipboard operation
geoip copied to clipboard

Use MaxMind Supplied Test DBs

Open mtodd opened this issue 13 years ago • 13 comments

I've contacted MaxMind and they've been gracious enough to provide us with test databases that we can distribute along with the client library to simplify testing. This pull includes these new files and slight refactoring to the tests.

This also includes a fix for #7 that only attempts lookups of region_name when region is found, which was previously causing segfaults.

mtodd avatar Jul 11 '12 16:07 mtodd

Awaiting some updated test DBs from the MaxMind contact (to get the tests passing) and some further details before merging.

mtodd avatar Jul 11 '12 16:07 mtodd

@mtodd, any news on the test DBs?

alindeman avatar Sep 15 '12 19:09 alindeman

While we await these test DBs from MaxMind, this gem crashes (see #7). Are the test DBs worth blocking a fix for a fatal error condition?

findchris avatar Sep 27 '12 18:09 findchris

Is everyone still using this gem? If segfault-fixes are not being merged on this project, is there another that you all are using? We've been using our own fork, but we thought that would be temporary.

findchris avatar Jan 30 '13 20:01 findchris

@mtodd, I don't feel like I have the time or expertise anymore to maintain this gem. Thoughts on inviting new maintainers from the folks submitting pull requests?

alindeman avatar Jan 30 '13 21:01 alindeman

@findchris, are you interested in maintaining the gem?

alindeman avatar Mar 01 '13 15:03 alindeman

@alindeman I would be interested, but I've been using the geoip-c gem, and unless there is a compelling reason to maintain this one, I'll probably pass.

findchris avatar Mar 01 '13 19:03 findchris

@alindeman I was clearly confused in my reply. This is the geoip-c gem, and I guess I thought I was responding to a different request.

That said, I am interested in maintaining this gem, as this gem deserves more attention.

findchris avatar Aug 06 '13 21:08 findchris

Right, this is the geoip-c gem. I'm :+1: to make you a committer. I'm not using this gem day-to-day anymore, so I'm not a good candidate to keep things moving. @mtodd has the final call though.

alindeman avatar Aug 06 '13 22:08 alindeman

Cool. Hopefully @mtodd will chime in here. By the way, @alindeman, are you using any geo data anymore, and if so, what gem are you using to access that data (out of curiosity)?

findchris avatar Aug 06 '13 22:08 findchris

By the way, @alindeman, are you using any geo data anymore, and if so, what gem are you using to access that data (out of curiosity)?

Ah, I'm just not currently working on a project that uses an IP -> geo database anymore. I believe the project itself is still using this gem, but other folks are working on it now.

alindeman avatar Aug 06 '13 22:08 alindeman

I'm very eager to find someone to take over maintaing this gem. I simply don't have the time or attention to properly care for it and know that that won't be changing any time soon.

mtodd avatar Aug 18 '13 06:08 mtodd

@mtodd Understood. Can I maintain the project or be a committer then?

findchris avatar Aug 19 '13 22:08 findchris