Skip to content

Replace HOST with BIND#120

Merged
dahlia merged 3 commits into
fedify-dev:mainfrom
ThisIsMissEm:fix/replace-host-with-bind
Feb 23, 2025
Merged

Replace HOST with BIND#120
dahlia merged 3 commits into
fedify-dev:mainfrom
ThisIsMissEm:fix/replace-host-with-bind

Conversation

@ThisIsMissEm

Copy link
Copy Markdown
Contributor

This fixes the issue with the container not listening on all interfaces by default (was a breaking change in #114), now BIND will only be used if it's set, otherwise it is undefined which means "listen on all interfaces")

@ThisIsMissEm ThisIsMissEm force-pushed the fix/replace-host-with-bind branch from 2a921e5 to c31d546 Compare February 22, 2025 15:59

@ThisIsMissEm ThisIsMissEm left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Pointing out that translations might need to be aligned.

Comment thread docs/src/content/docs/ja/install/env.mdx Outdated
Comment thread docs/src/content/docs/ko/install/env.mdx Outdated
Comment thread docs/src/content/docs/zh-cn/install/env.mdx Outdated
Comment thread .env.sample
LOG_QUERY=false

BEHIND_PROXY=false
LISTEN_PORT=3000

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@dahlia this is where that LISTEN_PORT came from! I knew it was something that was already there and I didn't invent it in #114, turns out it should've been PORT in the sample environment file.

@dahlia dahlia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Everything looks good except for translations! I left change suggestions for them.

Co-authored-by: Hong Minhee (洪 民憙) <hong@minhee.org>
@ThisIsMissEm

Copy link
Copy Markdown
Contributor Author

Updated!

@dahlia dahlia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@dahlia dahlia merged commit a4ea8b1 into fedify-dev:main Feb 23, 2025
dahlia added a commit that referenced this pull request Feb 23, 2025
@ThisIsMissEm ThisIsMissEm deleted the fix/replace-host-with-bind branch February 24, 2025 02:27
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