Skip to content

Send logs over tls#6

Merged
bsphere merged 3 commits intobsphere:masterfrom
StephenHynes7:use-secure-port
May 18, 2016
Merged

Send logs over tls#6
bsphere merged 3 commits intobsphere:masterfrom
StephenHynes7:use-secure-port

Conversation

@StephenHynes7
Copy link
Contributor

Should send over tls versus plain text.

@StephenHynes7
Copy link
Contributor Author

@bsphere can you take a look?

@bsphere
Copy link
Owner

bsphere commented May 17, 2016

Build is failing, please fix..

@jcftang-r7
Copy link
Contributor

This is an easy fix...

1z [jtang:~/sandbox/le_go] [sandbox] use-secure-port(+1/-1)* 11s ± git diff
diff --git a/le.go b/le.go
index 596db7d..3d80081 100644
--- a/le.go
+++ b/le.go
@@ -59,7 +59,7 @@ func (logger *Logger) Close() error {
 // Opens a TCP connection to logentries.com
 func (logger *Logger) openConnection() error {
        config := tls.Config{}
-       conn, err := tls.Dial("tcp", "data.logentries.com:443")
+       conn, err := tls.Dial("tcp", "data.logentries.com:443", &config)
        if err != nil {
                return err
        }

@StephenHynes7
Copy link
Contributor Author

Rebuilding now.

@StephenHynes7
Copy link
Contributor Author

@bsphere should be good now.

@caarlos0
Copy link
Contributor

Very nice!

@bsphere bsphere merged commit 763c0d6 into bsphere:master May 18, 2016
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.

4 participants