Propagate changes from main into development#880
Merged
Wikid82 merged 2 commits intodevelopmentfrom Mar 24, 2026
Merged
Conversation
Automated checksum update for GeoLite2-Country.mmdb database. Old: aa154fc6bcd712644de232a4abcdd07dac1f801308c0b6f93dbc2b375443da7b New: c6549807950f93f609d6433fa295fa517fbdec0ad975a4aafba69c136d5d2347 Auto-generated by: .github/workflows/update-geolite2.yml
chore(docker): update GeoLite2-Country.mmdb checksum
Contributor
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
Contributor
There was a problem hiding this comment.
Pull request overview
Automated sync PR to keep development aligned with main, updating the pinned checksum used to validate the downloaded GeoLite2 Country database during Docker image builds.
Changes:
- Update
GEOLITE2_COUNTRY_SHA256to match the current GeoLite2-Country.mmdb content used by the Docker build.
Comments suppressed due to low confidence (1)
Dockerfile:439
- The GeoLite2 database is downloaded from a mutable URL (GitHub
raw/download), and in the CI branch the file is accepted without any checksum verification. This can cause supply-chain risk (unverified artifact in CI builds) and also leads to frequent rebuild breakages when the upstream file changes and the pinned SHA becomes stale. Consider pinning the download to a specific release/tag/commit (immutable URL) and verifying the checksum in CI as well (even if with a shorter timeout).
ARG GEOLITE2_COUNTRY_SHA256=c6549807950f93f609d6433fa295fa517fbdec0ad975a4aafba69c136d5d2347
RUN mkdir -p /app/data/geoip && \
if [ -n "$CI" ]; then \
echo "⏱️ CI detected - quick download (10s timeout, no retries)"; \
if curl -fSL -m 10 "https://github.com/P3TERX/GeoLite.mmdb/raw/download/GeoLite2-Country.mmdb" \
-o /app/data/geoip/GeoLite2-Country.mmdb 2>/dev/null; then \
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated PR to propagate changes from main into development.
Triggered by push to main.