-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Relax restrictions on service names #1519
Comments
👍 I'll finally be able to build my
|
Also relevant: allowed characters in image tags, since those are also based on service name. Seems to be the same chars though (tags.go). |
+1 |
Is there anything new here? Will this change be part of the next RC or is it already baken into the current one? |
1.3 is in code freeze, so this will not be in the next release. |
Is there anything happening with support on this? I'm using 1.3 and it doesn't seem to be any different. Will there be a 1.3.1 or will we need to wait till 1.4 till anything other than [a-z0-9] is allowed for reasons of service/container/host compatibility/bug? |
This won't be in 1.3, or any 1.3.x release. 1.4 is the likely target. |
Closed by #1624. |
"Any good reason we can't allow _.- now?" There is a good reason for "_", since service names are used as host names in URLs : "Note that while upper and lower case letters are allowed in domain The labels must follow the rules for ARPANET host names. They must https://www.ietf.org/rfc/rfc1034.txt If your try to reach a tomcat served URL inside from a container to an other container (using his service name), tomcat will pop a "java.lang.IllegalArgumentException: The character [_] is never valid in a domain name." |
I wasted a day because I'm using underscore in my docker-compose. But somebody uses RFC (RFC 822) for his product. |
Now that we're not relying on a strict container name format, we should be able to allow more characters in service names, as has often been requested (see #799, #869, #941, #1300, #1384).
[a-zA-Z0-9_.-]
(daemon.go)[a-zA-Z0-9]
(service.py)Any good reason we can't allow
_.-
now?The text was updated successfully, but these errors were encountered: