Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for GeoIP2-ISP database #91

Closed
ejectck opened this issue Jul 23, 2016 · 9 comments
Closed

Add support for GeoIP2-ISP database #91

ejectck opened this issue Jul 23, 2016 · 9 comments

Comments

@ejectck
Copy link

ejectck commented Jul 23, 2016

As written in comment to geoip.rb "The path to the GeoLite2 database file which Logstash should use. Only City database is supported by now".

As for support Legacy database dropped in geoip, for me there is no way to get information from GeoIPASNum.dat (legacy) or GeoIP2-ISP.mmdb (new format) databases to get information about ASNum by IP.

Dropping support for Legacy and not adding support for new database is quite big problem for me.

imriz added a commit to myheritage/logstash-filter-geoip that referenced this issue Aug 7, 2016
@imriz
Copy link

imriz commented Aug 7, 2016

Hi,

I encountered this today too.
I've patched the plugin to allow you to use the ISP database.
See PR #94

I am currently using it as:

    geoip {
      source => "client_ip"
    }
    geoip {
      source => "client_ip"
      dbtype => "isp"
      merge => true
      database => "/etc/maxmind/GeoIP2-ISP.mmdb"
    }

imriz added a commit to myheritage/logstash-filter-geoip that referenced this issue Aug 7, 2016
imriz added a commit to myheritage/logstash-filter-geoip that referenced this issue Aug 8, 2016
imriz added a commit to myheritage/logstash-filter-geoip that referenced this issue Aug 8, 2016
@IngaFeick
Copy link

+1

1 similar comment
@briangruber
Copy link

+1

@tomsommer
Copy link

@sh4t
Copy link

sh4t commented Jan 20, 2017

They do have ISP-ASN database as .mmdb; just not available publicly to my knowledge. It is available in the legacy format, and .mmdb format to paying subscribers.

@SMAPPER
Copy link

SMAPPER commented Mar 3, 2017

Any idea of if/when this may be put into general availability?

@jordansissel
Copy link
Contributor

jordansissel commented Mar 3, 2017

@SMAPPER please try to not comment on the same issue in multiple places at the same time (reference: #96 (comment) and #91 (comment)).

If you want updates on this issue, I recommend watching this issue in Github:

image

@acchen97
Copy link

An update for those encountering this: we now have a plan for resolving ISP and ASN lookups. Feel free to provide additional feedback on this issue #110.

@acchen97
Copy link

Closing as this has been delivered with geoip filter v4.1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants