-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Revert Port environment variable workaround in Aspire Dashboard images #5190
Labels
Comments
[Triage] @joperezr, can you make this change when the correct fix is in Aspire? I didn't find a tracking issue for it. |
Yup, will do. |
@joperezr is this fixed in Aspire Dashboard yet? Please link a tracking issue from the Aspire repo. |
Hi, I don't know of a tracking issue for this in dotnet/aspire, but I'm fixing it here: dotnet/aspire#5941 It should be fixed for Aspire 9.0. |
Fixed in #5980 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Original issue: #5164
The Aspire Dashboard Dockerfiles currently contain a workaround for a port environment variable issue with the Aspire Dashboard:
dotnet-docker/src/aspire-dashboard/8.0/cbl-mariner-distroless/amd64/Dockerfile
Lines 31 to 32 in 02e6fbd
I did not find a tracking issue for this on the Aspire repo but presumably it will be fixed eventually. We want all port/URL environment variables to have the shorthand format:
dotnet-docker/src/runtime-deps/6.0/alpine3.18/amd64/Dockerfile
Line 5 in 32c6a5c
The text was updated successfully, but these errors were encountered: