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

Use a public CDN endpoint to get MaxMind GeoLite2 Data by default #3942

Closed
Tracked by #253
asifsmohammed opened this issue Jan 10, 2024 · 1 comment · Fixed by #4310
Closed
Tracked by #253

Use a public CDN endpoint to get MaxMind GeoLite2 Data by default #3942

asifsmohammed opened this issue Jan 10, 2024 · 1 comment · Fixed by #4310
Assignees
Milestone

Comments

@asifsmohammed
Copy link
Member

asifsmohammed commented Jan 10, 2024

Use a public CDN endpoint to improve the user experience and provide MaxMind GeoLite2 Database by default.

Configuration for using geoip can be as simple as this,

processor:
  - geoip:
      keys:
          - key:
              source: "/peer/ip"
              target: "target1"
@asifsmohammed asifsmohammed self-assigned this Jan 10, 2024
@asifsmohammed asifsmohammed added this to the v2.7 milestone Jan 10, 2024
@dlvenable
Copy link
Member

@asifsmohammed , Do you have a configuration proposal for the data-prepper-config.yaml file?

Also, a CDN is a server-side technology to push content close to users. Clients experience this as making a request to a URL. So I'd rephrase this proposal to "Use a configurable HTTP endpoint to get..."

dlvenable added a commit to dlvenable/data-prepper that referenced this issue Mar 21, 2024
…the correct defaults when loading. Resolves opensearch-project#3942

Signed-off-by: David Venable <dlv@amazon.com>
@dlvenable dlvenable self-assigned this Mar 21, 2024
dlvenable added a commit that referenced this issue Mar 21, 2024
…the correct defaults when loading. Resolves #3942 (#4310)

Signed-off-by: David Venable <dlv@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants