search

envoyproxy / envoy

Cloud-native high-performance edge/middle/service proxy
https://www.envoyproxy.io
Apache License 2.0
24.82k stars 4.77k forks source link

Supporting maxmind country db in geoip maxmind provider #30956

Open hwyuan opened 10 months ago

hwyuan commented 10 months ago

Title: Supporting maxmind country db in geoip maxmind provider

Description: Currently the geoip maxmind provider supports city, isp, and anonymous database files. In cases where only country-level information is needed, it's desirable to have the option of loading only the country database, because the country database is a lot smaller (5-6MB) than city database (~100MB) and thus can be more efficient.

[optional Relevant Links:] maxmind provider config api: https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/geoip_providers/maxmind/v3/maxmind.proto#envoy-v3-api-msg-extensions-geoip-providers-maxmind-v3-maxmindconfig

KBaichoo commented 10 months ago

cc @nezdolik

nezdolik commented 10 months ago

Sounds like a reasonable feature, @hwyuan would you submit a patch for it?

hwyuan commented 10 months ago

@nezdolik yea sure, will add a patch for this, thanks!

github-actions[bot] commented 9 months ago

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

github-actions[bot] commented 9 months ago

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted" or "no stalebot". Thank you for your contributions.

hwyuan commented 7 months ago

pining the thread, could we re-open this issue and PR cc @nezdolik