ldraney/landscaping-assistant
1
0
Fork 0
Code Issues 32 Pull requests 1 Projects Releases Packages Wiki Activity Actions 4

Fix broken images: proxy ActiveStorage through Rails instead of redirecting to internal MinIO URL #41

New issue
Closed
opened 2026-05-29 11:50:54 +00:00 by ldraney · 0 comments
ldraney commented 2026-05-29 11:50:54 +00:00
Owner
Copy link

Type

Bug

Lineage

Standalone bug fix — no parent plan or phase.

Repo

ldraney/landscaping-assistant

What Broke

Images uploaded via the Photos tab show as broken in the browser. ActiveStorage's default redirect mode sends a 302 to minio.minio.svc.cluster.local:9000, which is a cluster-internal address the browser can't resolve.

Repro Steps

  1. Upload a photo on the Photos tab
  2. View the upload show page (/uploads/:id)
  3. Image shows broken icon with alt text

Expected Behavior

Images display correctly on both the index (thumbnails) and show (full-size) pages.

Environment

Production — pal-e k8s cluster, MinIO in minio namespace, app in landscaping-assistant namespace.

Acceptance Criteria

  • Images display correctly on the uploads index and show pages
  • Thumbnails render on the index page
  • Full-size images render on the show page

Related

  • PR #36 introduced ActiveStorage + MinIO uploads
  • PR #384 (pal-e-platform) fixed MinIO NetworkPolicy to allow connections
### Type Bug ### Lineage Standalone bug fix — no parent plan or phase. ### Repo ldraney/landscaping-assistant ### What Broke Images uploaded via the Photos tab show as broken in the browser. ActiveStorage's default `redirect` mode sends a 302 to `minio.minio.svc.cluster.local:9000`, which is a cluster-internal address the browser can't resolve. ### Repro Steps 1. Upload a photo on the Photos tab 2. View the upload show page (`/uploads/:id`) 3. Image shows broken icon with alt text ### Expected Behavior Images display correctly on both the index (thumbnails) and show (full-size) pages. ### Environment Production — pal-e k8s cluster, MinIO in `minio` namespace, app in `landscaping-assistant` namespace. ### Acceptance Criteria - [ ] Images display correctly on the uploads index and show pages - [ ] Thumbnails render on the index page - [ ] Full-size images render on the show page ### Related - PR #36 introduced ActiveStorage + MinIO uploads - PR #384 (pal-e-platform) fixed MinIO NetworkPolicy to allow connections
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
docs/update-app-architecture
revert-213
move-other-crew-bottom-remove-schedule
feature/today-view-improvements
154-enable-assume-ssl
revert-145-person-icon
docs/user-stories-auth
docs/update-infra-pipeline-post-incident
fix-cache-step-blocking-deploy
fix-kaniko-registry-pull
deactivate-from-today
feature/quick-address-entry
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ldraney/landscaping-assistant#41
No description provided.