TUN-5597: Log session ID when session is terminated by edge

chungthuang · chungthuang · commit 74a30269634a · 2021-12-23T11:43:23.000Z
diff --git a/connection/quic.go b/connection/quic.go
@@ -198,10 +198,8 @@ func (q *QUICConnection) serveUDPSession(session *datagramsession.Session, close
 		} else {
 			q.closeUDPSession(ctx, session.ID, "terminated without error")
 		}
-		q.logger.Debug().Err(err).Str("sessionID", session.ID.String()).Msg("session terminated")
-		return
 	}
-	q.logger.Debug().Err(err).Msg("Session terminated by edge")
+	q.logger.Debug().Err(err).Str("sessionID", session.ID.String()).Msg("Session terminated")
 }
 
 // closeUDPSession first unregisters the session from session manager, then it tries to unregister from edge

Original file line number	Diff line number	Diff line change
`@@ -198,10 +198,8 @@ func (q QUICConnection) serveUDPSession(session datagramsession.Session, close`
`198`	`198`	`} else {`
`199`	`199`	`q.closeUDPSession(ctx, session.ID, "terminated without error")`
`200`	`200`	`}`
`201`		`- q.logger.Debug().Err(err).Str("sessionID", session.ID.String()).Msg("session terminated")`
`202`		`- return`
`203`	`201`	`}`
`204`		`- q.logger.Debug().Err(err).Msg("Session terminated by edge")`
	`202`	`+ q.logger.Debug().Err(err).Str("sessionID", session.ID.String()).Msg("Session terminated")`
`205`	`203`	`}`
`206`	`204`
`207`	`205`	`// closeUDPSession first unregisters the session from session manager, then it tries to unregister from edge`