Skip to content
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

ci: fix test-proxy #1599

Closed
wants to merge 7 commits into from
Closed

ci: fix test-proxy #1599

wants to merge 7 commits into from

Conversation

dscho
Copy link
Contributor

@dscho dscho commented Jan 31, 2024

According to https://hub.docker.com/r/ubuntu/squid, the proxy won't be available via http://squid.proxy:3128/ but instead via http://localhost:3128/.

I noticed this when the CI failed for #1598, for reasons unrelated to that PR, so I fixed it.

According to https://hub.docker.com/r/ubuntu/squid, the proxy won't be
available via http://squid.proxy:3128/ but instead via
http://localhost:3128/.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This reverts commit 0ebad4a.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Contributor Author

dscho commented Jan 31, 2024

Okay, I cannot figure it out, nor justify to continue working on this.

@dscho dscho closed this Jan 31, 2024
@dscho dscho deleted the fix-test-proxy branch January 31, 2024 14:56
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.

None yet

1 participant