fix(docker): add missing ipv6 all-interfaces binding

[Southclaws]

Resolves #8437

Motivation and Context

See the above linked issue and related Fly.io forum post for details.

Checklist

My PR contains...

• No code changes (src/ is unmodified: changes to documentation, CI, metadata, etc.)
• Dependency changes (any modification to dependencies in package.json)
• Bug fixes (non-breaking change which fixes an issue)
• Improvements (misc. changes to existing features)
• Features (non-breaking change which adds functionality)

My changes...

• are breaking changes to a public API (config options, System API, major UI change, etc).
• are breaking changes to a private API (Redux, component props, utility functions, etc.).
• are breaking changes to a developer API (npm script behavior changes, new dev system dependencies, etc).
• are not breaking changes.

Documentation

• My changes do not require a change to the project documentation.
• My changes require a change to the project documentation.
• If yes to above: I have updated the documentation accordingly.

Automated tests

• My changes can not or do not need to be tested.
• My changes can and should be tested by unit and/or integration tests.
• If yes to above: I have added tests to cover my changes.
• If yes to above: I have taken care to cover edge cases in my tests.
• All new and existing tests passed.

[char0n]

Hi @Southclaws,

Thanks for the PR!

Here are some of my research notes:

NOTE1:

As of Nov. 2021 with Nginx latest (from the official repo) e.g. on Ubuntu 18.04 or 20.04, for regular (=not the default) Nginx vhosts this is what works for both IPv4 & IPv6 traffic:

listen [::]:80;

NOTE2:

starting with Nginx 1.3.4, the ipv6only parameter defaults to on, as we don't want to translate ipv4 to ipv6

swagger-ui/docker/docker-entrypoint.d/40-swagger-ui.sh

Line 44 in ec51dc3

sed -i "s|8080|${PORT}|g" $NGINX_CONF

will properly replace the port for ipv6 as well.

Resources:

• https://serverfault.com/questions/638367/do-you-need-separate-ipv4-and-ipv6-listen-directives-in-nginx

[char0n]

LGTM! Thank you for contributing.

[char0n]

I can confirm that the change worked locally with following commands:

$ docker build . -t swagger-api/swagger-ui:latest
$ docker run -p 80:8080 swagger-api/swagger-ui:latest

And opening http://localhost:80 in my browser.