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

Support bracket notation for IPv6 port mappings #7001

Closed
chris13524 opened this issue Nov 5, 2019 · 3 comments
Closed

Support bracket notation for IPv6 port mappings #7001

chris13524 opened this issue Nov 5, 2019 · 3 comments

Comments

@chris13524
Copy link

I think it would be a good idea to support the bracket notation for IPv6 port mappings. That is, you should be able to do this:

ports:
  - "[::1]:80:80"

I'm aware you can simply omit the brackets to do this currently, however I think supporting the bracket notation would be a good idea.

I believe this for several reasons:

  • it is standard to use brackets when writing out IPv6 addresses and port numbers
  • the docker run command uses this bracket format for port mappings
  • applications such as web browsers and curl use brackets
  • it is easier to parse with the eyes
  • for some (like myself), it is intuitive to write out port mappings using the brackets just to be greeted by this confusing error message:
    ERROR: The Compose file './docker-compose.yml' is invalid because:
    services.nginx.ports contains an invalid type, it should be a number, or an object
    
    I've done this several times and after a few minutes of Googling come across this issue which reminds me.
@jcsirot
Copy link

jcsirot commented Nov 6, 2019

Hello @chris13524
Thank you for your feature suggestion. We are going to investigate the possibility to integrate it in a future release.

angristan added a commit to angristan/nano-docs that referenced this issue Feb 1, 2021
The bracket notation for IPv6 doesn't seem to be supported: docker/compose#7001.
angristan added a commit to angristan/nano-docs that referenced this issue Feb 1, 2021
The bracket notation for IPv6 doesn't seem to be supported: docker/compose#7001.
zhyatt pushed a commit to nanocurrency/nano-docs that referenced this issue Feb 8, 2021
The bracket notation for IPv6 doesn't seem to be supported: docker/compose#7001.
@ndeloof
Copy link
Contributor

ndeloof commented Jun 17, 2021

Actually implemented separately

@ndeloof ndeloof closed this as completed Jun 17, 2021
@ndeloof
Copy link
Contributor

ndeloof commented Jun 17, 2021

docker/docker-py#2784

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants