diff --git a/congestion/rtt_stats.go b/congestion/rtt_stats.go index 3dc89e2fe..6f95e4925 100644 --- a/congestion/rtt_stats.go +++ b/congestion/rtt_stats.go @@ -84,7 +84,7 @@ func (r *RTTStats) SetRecentMinRTTwindow(recentMinRTTwindow time.Duration) { // UpdateRTT updates the RTT based on a new sample. func (r *RTTStats) UpdateRTT(sendDelta, ackDelay time.Duration, now time.Time) { if sendDelta == math.MaxInt64 || sendDelta <= 0 { - utils.Infof("Ignoring measured sendDelta, because it's is either infinite, zero, or negative: %d\n", sendDelta/time.Microsecond) + utils.Infof("Ignoring measured sendDelta, because it's is either infinite, zero, or negative: %d", sendDelta/time.Microsecond) return } diff --git a/h2quic/response_writer.go b/h2quic/response_writer.go index 7ece2ab2d..3499b4e1b 100644 --- a/h2quic/response_writer.go +++ b/h2quic/response_writer.go @@ -47,7 +47,7 @@ func (w *responseWriter) WriteHeader(status int) { enc.WriteField(hpack.HeaderField{Name: k, Value: v[0]}) } - utils.Infof("Responding with %d %#v\n", status, w.header) + utils.Infof("Responding with %d %#v", status, w.header) h2framer := http2.NewFramer(w.headerStream, nil) err := h2framer.WriteHeaders(http2.HeadersFrameParam{ StreamID: uint32(w.dataStreamID), @@ -70,7 +70,7 @@ func (w *responseWriter) Write(p []byte) (int, error) { var err error w.dataStream, err = w.session.NewStream(w.dataStreamID) if err != nil { - return 0, fmt.Errorf("error creating data stream: %s\n", err.Error()) + return 0, fmt.Errorf("error creating data stream: %s", err.Error()) } } return w.dataStream.Write(p) diff --git a/h2quic/server.go b/h2quic/server.go index 7fd053ab3..6b8628b6a 100644 --- a/h2quic/server.go +++ b/h2quic/server.go @@ -48,7 +48,7 @@ func (s *Server) handleStream(session *quic.Session, headerStream utils.Stream) go func() { for { if err := s.handleRequest(session, headerStream, hpackDecoder, h2framer); err != nil { - utils.Errorf("error handling h2 request: %s\n", err.Error()) + utils.Errorf("error handling h2 request: %s", err.Error()) return } } @@ -66,7 +66,7 @@ func (s *Server) handleRequest(session *quic.Session, headerStream utils.Stream, } headers, err := hpackDecoder.DecodeFull(h2headersFrame.HeaderBlockFragment()) if err != nil { - utils.Errorf("invalid http2 headers encoding: %s\n", err.Error()) + utils.Errorf("invalid http2 headers encoding: %s", err.Error()) return err } @@ -74,7 +74,7 @@ func (s *Server) handleRequest(session *quic.Session, headerStream utils.Stream, if err != nil { return err } - utils.Infof("Request: %#v\n", req) + utils.Infof("Request: %#v", req) responseWriter := &responseWriter{ header: http.Header{}, diff --git a/server.go b/server.go index 526d93c82..5b0da8712 100644 --- a/server.go +++ b/server.go @@ -93,7 +93,7 @@ func (s *Server) handlePacket(conn *net.UDPConn, remoteAddr *net.UDPAddr, packet // Send Version Negotiation Packet if the client is speaking a different protocol version if publicHeader.VersionFlag && !protocol.IsSupportedVersion(publicHeader.VersionNumber) { - utils.Infof("Client offered version %d, sending VersionNegotiationPacket\n", publicHeader.VersionNumber) + utils.Infof("Client offered version %d, sending VersionNegotiationPacket", publicHeader.VersionNumber) _, err = conn.WriteToUDP(composeVersionNegotiation(publicHeader.ConnectionID), remoteAddr) if err != nil { return err @@ -103,7 +103,7 @@ func (s *Server) handlePacket(conn *net.UDPConn, remoteAddr *net.UDPAddr, packet session, ok := s.sessions[publicHeader.ConnectionID] if !ok { - utils.Infof("Serving new connection: %d from %v\n", publicHeader.ConnectionID, remoteAddr) + utils.Infof("Serving new connection: %d from %v", publicHeader.ConnectionID, remoteAddr) session = s.newSession( &udpConn{conn: conn, currentAddr: remoteAddr}, publicHeader.VersionNumber, diff --git a/session.go b/session.go index 1c20ac643..ab814554b 100644 --- a/session.go +++ b/session.go @@ -115,7 +115,7 @@ func (s *Session) Run() { case ackhandler.ErrMapAccess: s.Close(err, true) // TODO: sent correct error code here case errRstStreamOnInvalidStream: - utils.Errorf("Ignoring error in session: %s\n", err.Error()) + utils.Errorf("Ignoring error in session: %s", err.Error()) default: s.Close(err, true) } @@ -133,7 +133,7 @@ func (s *Session) handlePacket(remoteAddr interface{}, publicHeader *PublicHeade publicHeader.PacketNumber, ) s.lastRcvdPacketNumber = publicHeader.PacketNumber - utils.Infof("<- Reading packet 0x%x (%d bytes) for connection %x\n", publicHeader.PacketNumber, r.Size(), publicHeader.ConnectionID) + utils.Infof("<- Reading packet 0x%x (%d bytes) for connection %x", publicHeader.PacketNumber, r.Size(), publicHeader.ConnectionID) // TODO: Only do this after authenticating s.conn.setCurrentRemoteAddr(remoteAddr) @@ -156,31 +156,31 @@ func (s *Session) handlePacket(remoteAddr interface{}, publicHeader *PublicHeade var err error switch frame := ff.(type) { case *frames.StreamFrame: - utils.Debugf("\t<- &frames.StreamFrame{StreamID: %d, FinBit: %t, Offset: %d}\n", frame.StreamID, frame.FinBit, frame.Offset) + utils.Debugf("\t<- &frames.StreamFrame{StreamID: %d, FinBit: %t, Offset: %d}", frame.StreamID, frame.FinBit, frame.Offset) err = s.handleStreamFrame(frame) case *frames.AckFrame: var duration time.Duration duration, err = s.sentPacketHandler.ReceivedAck(frame) s.rttStats.UpdateRTT(duration, frame.DelayTime, time.Now()) - utils.Debugf("\t<- %#v\n", frame) - utils.Debugf("\tEstimated RTT: %dms\n", s.rttStats.SmoothedRTT()/time.Millisecond) + utils.Debugf("\t<- %#v", frame) + utils.Debugf("\tEstimated RTT: %dms", s.rttStats.SmoothedRTT()/time.Millisecond) // ToDo: send right error in ConnectionClose frame case *frames.ConnectionCloseFrame: - utils.Debugf("\t<- %#v\n", frame) + utils.Debugf("\t<- %#v", frame) s.Close(nil, false) case *frames.StopWaitingFrame: - utils.Debugf("\t<- %#v\n", frame) + utils.Debugf("\t<- %#v", frame) err = s.receivedPacketHandler.ReceivedStopWaiting(frame) case *frames.RstStreamFrame: err = s.handleRstStreamFrame(frame) - utils.Debugf("\t<- %#v\n", frame) + utils.Debugf("\t<- %#v", frame) case *frames.WindowUpdateFrame: - utils.Debugf("\t<- %#v\n", frame) + utils.Debugf("\t<- %#v", frame) err = s.handleWindowUpdateFrame(frame) case *frames.BlockedFrame: - utils.Infof("BLOCKED frame received for connection %x stream %d\n", s.connectionID, frame.StreamID) + utils.Infof("BLOCKED frame received for connection %x stream %d", s.connectionID, frame.StreamID) case *frames.PingFrame: - utils.Debugf("\t<- %#v\n", frame) + utils.Debugf("\t<- %#v", frame) default: panic("unexpected frame type") } @@ -268,7 +268,7 @@ func (s *Session) Close(e error, sendConnectionClose bool) error { if e == nil { e = protocol.NewQuicError(errorcodes.QUIC_PEER_GOING_AWAY, "peer going away") } - utils.Errorf("Closing session with error: %s\n", e.Error()) + utils.Errorf("Closing session with error: %s", e.Error()) errorCode := protocol.ErrorCode(1) reasonPhrase := e.Error() quicError, ok := e.(*protocol.QuicError) @@ -311,7 +311,7 @@ func (s *Session) sendPacket() error { // TODO: handle multiple packets retransmissions retransmitPacket := s.sentPacketHandler.DequeuePacketForRetransmission() if retransmitPacket != nil { - utils.Infof("\tQueueing retransmission for packet 0x%x\n", retransmitPacket.PacketNumber) + utils.Infof("\tQueueing retransmission for packet 0x%x", retransmitPacket.PacketNumber) s.stopWaitingManager.RegisterPacketForRetransmission(retransmitPacket) // resend the frames that were in the packet controlFrames = append(controlFrames, retransmitPacket.GetControlFramesForRetransmission()...) @@ -347,12 +347,12 @@ func (s *Session) sendPacket() error { s.stopWaitingManager.SentStopWaitingWithPacket(packet.number) - utils.Infof("-> Sending packet 0x%x (%d bytes)\n", packet.number, len(packet.raw)) + utils.Infof("-> Sending packet 0x%x (%d bytes)", packet.number, len(packet.raw)) for _, frame := range packet.frames { if streamFrame, isStreamFrame := frame.(*frames.StreamFrame); isStreamFrame { - utils.Debugf("\t-> &frames.StreamFrame{StreamID: %d, FinBit: %t, Offset: 0x%x, Data length: 0x%x, Offset + Data length: 0x%x}\n", streamFrame.StreamID, streamFrame.FinBit, streamFrame.Offset, len(streamFrame.Data), streamFrame.Offset+uint64(len(streamFrame.Data))) + utils.Debugf("\t-> &frames.StreamFrame{StreamID: %d, FinBit: %t, Offset: 0x%x, Data length: 0x%x, Offset + Data length: 0x%x}", streamFrame.StreamID, streamFrame.FinBit, streamFrame.Offset, len(streamFrame.Data), streamFrame.Offset+uint64(len(streamFrame.Data))) } else { - utils.Debugf("\t-> %#v\n", frame) + utils.Debugf("\t-> %#v", frame) } } @@ -399,7 +399,7 @@ func (s *Session) garbageCollectStreams() { } func (s *Session) sendPublicReset(rejectedPacketNumber protocol.PacketNumber) error { - utils.Infof("Sending public reset for connection %x, packet number %d\n", s.connectionID, rejectedPacketNumber) + utils.Infof("Sending public reset for connection %x, packet number %d", s.connectionID, rejectedPacketNumber) packet := &publicResetPacket{ connectionID: s.connectionID, rejectedPacketNumber: rejectedPacketNumber,