Skip to content

fix: port of imageproxy was wrong#2269

Open
tinect (tinect) wants to merge 1 commit intoshopware:mainfrom
tinect:patch-7
Open

fix: port of imageproxy was wrong#2269
tinect (tinect) wants to merge 1 commit intoshopware:mainfrom
tinect:patch-7

Conversation

@tinect
Copy link
Copy Markdown
Contributor

Summary

the imageproxy starts at port 8000

Related links

Checklist

  • I reviewed affected links, code samples, and cross-references, including PageRef references where relevant.
  • I added or updated redirects in .gitbook.yaml if pages were moved, renamed, or deleted.
  • I updated .wordlist.txt (and sorted it) if spellcheck flags new legitimate terms.
  • Any required dependent changes in downstream modules have already been merged and published.
  • This pull request is ready for review.

Notes

Copilot AI review requested due to automatic review settings April 27, 2026 11:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the installation documentation to reflect the correct internal port for the imageproxy container so the proxy can be reached via the intended host port.

Changes:

  • Update the imageproxy Docker Compose port mapping from 8050:80 to 8050:8000.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 185 to +189
services:
imageproxy:
image: ghcr.io/shopwarelabs/devcontainer/image-proxy
ports:
- "8050:80"
- "8050:8000"
Copy link

Copilot AI Apr 27, 2026

Choose a reason for hiding this comment

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

The YAML example becomes invalid because the ports list item is not indented under ports: (it currently starts at the beginning of the line). Please indent the - "8050:8000" line to be a child of ports: (and ideally normalize indentation for the whole services/imageproxy block so the snippet can be copy-pasted into compose.override.yaml).

Copilot uses AI. Check for mistakes.
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.

2 participants