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

Docker: fix exposed port to be 8071. #1596

Merged
merged 1 commit into from
Dec 27, 2024
Merged

Docker: fix exposed port to be 8071. #1596

merged 1 commit into from
Dec 27, 2024

Conversation

tasn
Copy link
Member

@tasn tasn commented Dec 27, 2024

I don't think exposing 8080 ever did anything as the server is listening on 8071. It looks like a customer has been facing issues with their docker setup not letting them map the port because it doesn't match the exposed from the dockerfile. I'm not sure how it works for everyone else but not here though anyhow it should be fixed.

Ref: https://x.com/0xblacklight/status/1872430388158533871

@tasn tasn requested a review from a team as a code owner December 27, 2024 01:07
I don't think exposing 8080 ever did anything as the server is listening on 8071.
It looks like a customer has been facing issues with their docker setup
not letting them map the port because it doesn't match the exposed from
the dockerfile. I'm not sure how it works for everyone else but not here
though anyhow it should be fixed.

Ref: https://x.com/0xblacklight/status/1872430388158533871
@tasn tasn force-pushed the tom/fix-docker-port branch from 956d8af to ae66286 Compare December 27, 2024 01:09
Copy link
Contributor

@svix-onelson svix-onelson left a comment

Choose a reason for hiding this comment

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

👍🏻

@K-Mistele
Copy link

Thanks for the quick fix, guys!

@tasn tasn merged commit 67fcc20 into main Dec 27, 2024
6 checks passed
@tasn tasn deleted the tom/fix-docker-port branch December 27, 2024 14:17
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