Skip to content

Conversation

@sjmiller609
Copy link
Collaborator

@sjmiller609 sjmiller609 commented Jan 27, 2026

Note

Fixes image name handling by decoding URL-encoded path params in ResolveResource (uses url.PathUnescape before calling resolvers). This ensures names like docker.io/library/nginx:alpine work when passed as %2F/%3A encoded segments.

  • Update lib/middleware/resolve.go to decode {name} for /images/{name} prior to resolution
  • Add lib/middleware/resolve_test.go to verify resolver receives decoded names across multiple cases
  • Extend cmd/api/main_test.go with end-to-end tests (via oapi.HandlerFromMux) asserting handlers get decoded image names and avoid 400s

Written by Cursor Bugbot for commit 8760323. This will update automatically on new commits. Configure here.

@sjmiller609 sjmiller609 merged commit da690d7 into main Jan 27, 2026
4 checks passed
@sjmiller609 sjmiller609 deleted the more-image-parsing branch January 27, 2026 18:37
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.

3 participants