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

syslog-address with tcp+tls protocol rejected by docker run #20042

Closed
jbergknoff opened this issue Feb 5, 2016 · 1 comment
Closed

syslog-address with tcp+tls protocol rejected by docker run #20042

jbergknoff opened this issue Feb 5, 2016 · 1 comment
Assignees
Labels
area/logging priority/P2 Normal priority: default priority applied.
Milestone

Comments

@jbergknoff
Copy link

I see the following behavior trying to use the new tcp+tls transport for the syslog log driver:

$ docker run --rm --log-driver=syslog --log-opt syslog-address=tcp+tls://logs.papertrailapp.com:1234 alpine:3.3 echo hello world
docker: Error response from daemon: syslog-address should be in form proto://address, got tcp+tls://logs.papertrailapp.com:1234.
$ docker -v
Docker version 1.10.0, build 590d5108

It looks like the relevant code is here: urlutil.

@thaJeztah
Copy link
Member

This was added in #18998, looks indeed that tcp+tls was added there, but the validation was not updated.

ping @calavera

@thaJeztah thaJeztah added this to the 1.10.1 milestone Feb 8, 2016
@tiborvass tiborvass added the priority/P2 Normal priority: default priority applied. label Feb 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logging priority/P2 Normal priority: default priority applied.
Projects
None yet
Development

No branches or pull requests

3 participants