Repository navigation
Add optional native MaxMind geolocation - #1499
Draft
ChristianPavilonis wants to merge 1 commit into
Draft
ChristianPavilonis wants to merge 1 commit into
ChristianPavilonis wants to merge 1 commit into
Conversation
Expose EdgeZero's local GeoLite provider through the native boot profile and Docker feature selection. Keep it default-off and reuse checked reader-IP resolution without a vendor-specific proxy parser. Add synthetic startup coverage, an opt-in operator snapshot test, CI feature combinations, and deployment and validation documentation.
This branch has not been deployed
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.
Summary
geo-maxmindfeature.This is stacked on
rc/edgezero-431-demo, the release candidate in #1424, to keep the diff limited to MaxMind support. EdgeZero remains pinned tofda87556f5aab160711eb5244c1d1065a6bb48d6.Changes
Cargo.tomlandCargo.lockcrates/trusted-server-adapter-axum/src/native_geo.rsmaxmind, the database path, and explicit reader-IP expectationDockerfileand.github/workflows/test.ymldocs/guide/native-rc-demo.mdandnative-rc-validation.mdRelated issues
No Trusted Server issue is linked yet. Upstream follow-up: IABTechLab/edgezero#438. This PR does not close it.
Spec
No separate agent-reviewed feature spec is linked yet. The implementation and operator contract are documented in
docs/guide/native-rc-demo.md; this is not a substitute for the required spec. Keep this PR in draft pending the issue/spec requirements and review readiness.Test plan
The following checks passed on the implementation before this commit. Before publishing, the current Rust/build inputs and synthetic fixtures were verified against the recorded SHA-256 manifest; no tested inputs changed. Tests were not rerun solely for committing.
geo-maxmind: 101 passed, 3 ignoredgeo-maxmind,geo-cloudfront: 103 passed, 3 ignored-D warnings, plus rustdocgit diff --checkDetailed results and remaining limits are in
docs/guide/native-rc-validation.md. The operator dataset stayed outside the repository and Docker build context. Local release readiness was about 2.6 seconds; peak startup RSS was not measured. Upstream's documented unused-metadata validation limitation remains.Checklist