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

Merged
merged 5 commits into from
Feb 24, 2025

Conversation

saalim-mushtaq
Copy link
Collaborator

No description provided.

@saalim-mushtaq
Copy link
Collaborator Author

@adnan-kamili

@@ -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

@adnan-kamili adnan-kamili merged commit a125bf1 into master Feb 24, 2025
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