Merge branch 'id-fix-the-apdex-metric' into 'main'

ashmckenzie · Igor Drozdov · ashmckenzie · commit be932e8d25e6 · 2023-08-08T08:07:31.000Z
Fix the established session metric See merge request https://gitlab.com/gitlab-org/gitlab-shell/-/merge_requests/826 Merged-by: Ash McKenzie <amckenzie@gitlab.com> Approved-by: Ash McKenzie <amckenzie@gitlab.com> Co-authored-by: Igor Drozdov <idrozdov@gitlab.com>
diff --git a/internal/sshd/sshd.go b/internal/sshd/sshd.go
@@ -203,7 +203,7 @@ func (s *Server) handleConn(ctx context.Context, nconn net.Conn) {
 			gitlabKeyId:         sconn.Permissions.Extensions["key-id"],
 			gitlabKrb5Principal: sconn.Permissions.Extensions["krb5principal"],
 			remoteAddr:          remoteAddr,
-			started:             started,
+			started:             time.Now(),
 		}
 
 		var err error

Original file line number	Diff line number	Diff line change
`@@ -203,7 +203,7 @@ func (s *Server) handleConn(ctx context.Context, nconn net.Conn) {`
`203`	`203`	`gitlabKeyId: sconn.Permissions.Extensions["key-id"],`
`204`	`204`	`gitlabKrb5Principal: sconn.Permissions.Extensions["krb5principal"],`
`205`	`205`	`remoteAddr: remoteAddr,`
`206`		`- started: started,`
	`206`	`+ started: time.Now(),`
`207`	`207`	`}`
`208`	`208`
`209`	`209`	`var err error`