Skip to content

fix(security): bump geo to 0.3.5 - #37

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/geo-0-3-5
Nov 6, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
fix/geo-0-3-5

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Bumps appwrite/geo from 0.3.2 to 0.3.5, built on appwrite/base 2.2.9 by appwrite/docker-geo#14. Part of appwrite-labs/cloud#7351.

0.3.3 and 0.3.4 are included: they allow utopia-php/dsn 1.0 and read the listen port from GEO_PORT. The chart does not set GEO_PORT, so the service stays on port 80.

Tested: the 0.3.5 source built locally passes its 9 E2E tests, and serves /v1/health with the settings this chart uses: read-only root filesystem, user 82, all capabilities dropped and the lowered unprivileged port range. The chart install in this repo's CI covers the published image.

The image was built on appwrite/base 1.1.1, which carries most of the fixable critical and high Trivy findings against this chart.
@appwrite

appwrite Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The chart now defaults the Geo service image to appwrite/geo:0.3.5. The image inventory, generated values documentation, optional-components guide, and changelog reflect the update; the deployment continues to use port 80.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 5
File Change
charts/appwrite/values.yaml Updates the default Geo image tag to 0.3.5.
charts/appwrite/Chart.yaml Updates the Geo image in the chart image inventory.
charts/appwrite/README.md Regenerates the documented Geo image tag.
docs/optional-components.md Updates the documented Geo image reference.
CHANGELOG.md Records the Geo image security update.

Reviewed 524b94f · Details · Comment @appwrite review to re-run, or mention @appwrite with a question.

@appwrite appwrite Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 71ab08d into main Nov 6, 2026
21 checks passed
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.

1 participant