The following pull request was submitted through Github.
It can be accessed and reviewed at: https://github.com/lxc/lxd/pull/4743

This e-mail was sent by the LXC bot, direct replies will not reach the author
unless they happen to be subscribed to this list.

=== Description (from pull-request) ===
This string

&{%!s(*os.file=&{{{1 0 0} -1 {0} <nil> 1 true true true true} master <nil> false false})}

is not helpful at all so stop dumping it to the log. It is just irritating.

Signed-off-by: Christian Brauner <[email protected]>
From 69e842e1193e59908451b19d5d1673cc3891852d Mon Sep 17 00:00:00 2001
From: Christian Brauner <[email protected]>
Date: Fri, 6 Jul 2018 01:27:29 +0200
Subject: [PATCH] network: do not print writer struct on error

This string

&{%!s(*os.file=&{{{1 0 0} -1 {0} <nil> 1 true true true true} master <nil> 
false false})}

is not helpful at all so stop dumping it to the log. It is just irritating.

Signed-off-by: Christian Brauner <[email protected]>
---
 shared/network.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/shared/network.go b/shared/network.go
index f47f75cea..88fa4c3cf 100644
--- a/shared/network.go
+++ b/shared/network.go
@@ -325,7 +325,7 @@ func defaultWriter(conn *websocket.Conn, w io.WriteCloser, 
writeDone chan<- bool
        for {
                mt, r, err := conn.NextReader()
                if err != nil {
-                       logger.Debugf("Got error getting next reader %s, %s", 
err, w)
+                       logger.Debugf("Got error getting next reader %s", err)
                        break
                }
 
_______________________________________________
lxc-devel mailing list
[email protected]
http://lists.linuxcontainers.org/listinfo/lxc-devel

Reply via email to