Skip to content

Conversation

@OE1KHZ
Copy link
Contributor

@OE1KHZ OE1KHZ commented Nov 20, 2025

This simple fix makes running this fork of synapse-admin on ipv4-only systems possible.
Currently, running synapse-admin on non-ipv6 systems throws the following error:

synapse-admin  | 2025-11-20T15:42:48.214133Z ERROR static_web_server::server: server failed to start up: failed to bind to [::]:80 address
synapse-admin  | 
synapse-admin  | Caused by:
synapse-admin  |     Address family not supported by protocol (os error 97)

As mentioned in this issue in static-web-server passing the environment variable SERVER_HOST=0.0.0.0 overrides the default (SERVER_HOST=[::]) and therefore fixes this issue :)

@aine-etke
Copy link
Member

Thank you for contribution!

Unfortunately, from the linked issue it sounds like such adjustment will break compatibility with IPv6-only systems, and may impact dual-stack systems as well.

Would you mind instead of setting the default value in the Dockerfile, mention that workaround in the readme?

@OE1KHZ
Copy link
Contributor Author

OE1KHZ commented Nov 20, 2025

Ahhh, yes sure. How have i not thought about that...

@OE1KHZ OE1KHZ force-pushed the patch-1 branch 3 times, most recently from e04f8a9 to cdd4a50 Compare November 21, 2025 14:05
@OE1KHZ
Copy link
Contributor Author

OE1KHZ commented Nov 21, 2025

Ok sorry for the force pushes but everything i messed up is gone, since i squashed and pushed the commits.

@OE1KHZ OE1KHZ changed the title fix synapse-admin on ipv6-disabled systems fix running on ipv4-only systems Nov 21, 2025
Copy link
Member

@aine-etke aine-etke left a comment

Choose a reason for hiding this comment

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

Thank you! Added a few notes - please take a look whenever you have a chance

@OE1KHZ
Copy link
Contributor Author

OE1KHZ commented Nov 22, 2025

Sorry for the mess. I thought the only way to set environment variables was to set them during build time. That's why I chose to use an build argument which gets saved in a environment variable. I just found out that you can set environment variables after building via -e flag.

@aine-etke
Copy link
Member

Thank you for handling it!

Yes, env vars can be set at run time with -e and --env-file args

@aine-etke aine-etke merged commit c5448d0 into etkecc:main Nov 23, 2025
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