Ver código fonte

return when error happens on listener close

Signed-off-by: Harry Zhang <harryz@hyper.sh>
Harry Zhang 8 anos atrás
pai
commit
390fe752a2
1 arquivos alterados com 1 adições e 3 exclusões
  1. 1 3
      pkg/listeners/listeners_unix.go

+ 1 - 3
pkg/listeners/listeners_unix.go

@@ -8,7 +8,6 @@ import (
 	"net"
 	"net"
 	"strconv"
 	"strconv"
 
 
-	"github.com/Sirupsen/logrus"
 	"github.com/coreos/go-systemd/activation"
 	"github.com/coreos/go-systemd/activation"
 	"github.com/docker/go-connections/sockets"
 	"github.com/docker/go-connections/sockets"
 )
 )
@@ -86,8 +85,7 @@ func listenFD(addr string, tlsConfig *tls.Config) ([]net.Listener, error) {
 			continue
 			continue
 		}
 		}
 		if err := ls.Close(); err != nil {
 		if err := ls.Close(); err != nil {
-			// TODO: We shouldn't log inside a library. Remove this or error out.
-			logrus.Errorf("failed to close systemd activated file: fd %d: %v", fdOffset+3, err)
+			return nil, fmt.Errorf("failed to close systemd activated file: fd %d: %v", fdOffset+3, err)
 		}
 		}
 	}
 	}
 	return []net.Listener{listeners[fdOffset]}, nil
 	return []net.Listener{listeners[fdOffset]}, nil