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

chore: add maxmind env var for geoip #13

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

saalim-mushtaq
Copy link
Collaborator

No description provided.

@saalim-mushtaq
Copy link
Collaborator Author

@adnan-kamili

@@ -23,6 +23,9 @@ FILE_STORE_SECRET_KEY=1password

GOOGLE_CLIENT_ID=PASTE_GOOGLE_CLIENT_ID

# GeoIP MaxMind
MAXMIND_LICENSE_KEY=YOUR_LICENSE_KEY_HERE

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MAXMIND_ACCOUNT_ID is missing @saalim-mushtaq

Copy link
Contributor

@adnan-kamili adnan-kamili Feb 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MaxMind GeoIP

MAXMIND_ACCOUNT_ID =PASTE_MAXMIND_ACCOUNT_ID
MAXMIND_LICENSE_KEY=PASTE_MAXMIND_LICENSE_KEY

@@ -110,6 +113,8 @@ services:
REDIS_URL: # redis://cache:6379 - not required for single machine deployment
GEOIPSERVER_URL: http://geoip:8080/json
GOOGLE_CLIENT_ID: ${GOOGLE_CLIENT_ID}
MAXMIND_LICENSE_KEY: ${MAXMIND_LICENSE_KEY}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MAXMIND_LICENSEKEY: ${MAXMIND_LICENSE_KEY}
MAXMIND_ACCOUNTID: ${MAXMIND_ACCOUNT_ID}

@saalim-mushtaq update here web api service has slightly different name for this env

.env Outdated
@@ -23,6 +23,10 @@ FILE_STORE_SECRET_KEY=1password

GOOGLE_CLIENT_ID=PASTE_GOOGLE_CLIENT_ID

# GeoIP MaxMind
MAXMIND_ACCOUNT_ID =PASTE_MAXMIND_ACCOUNT_ID
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@saalim-mushtaq space after MAXMIND_ACCOUNT_ID env

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

Successfully merging this pull request may close these issues.

2 participants