fix: cli client (#92)

* fix: cli client

* fix: print shutdown error correctly

pkg/client/rp/cli/cli.go

@@ -13,13 +13,14 @@ const (
 	loginPath = "/login"
 )
 
-func CodeFlow(relyingParty rp.RelyingParty, callbackPath, port string, stateProvider func() string) *oidc.Tokens {
+func CodeFlow(ctx context.Context, relyingParty rp.RelyingParty, callbackPath, port string, stateProvider func() string) *oidc.Tokens {
-	ctx, cancel := context.WithCancel(context.Background())
+	codeflowCtx, codeflowCancel := context.WithCancel(ctx)
-	defer cancel()
+	defer codeflowCancel()
+
+	tokenChan := make(chan *oidc.Tokens, 1)
 
-	var token *oidc.Tokens
 	callback := func(w http.ResponseWriter, r *http.Request, tokens *oidc.Tokens, state string) {
-		token = tokens
+		tokenChan <- tokens
 		msg := "<p><strong>Success!</strong></p>"
 		msg = msg + "<p>You are authenticated and can now return to the CLI.</p>"
 		w.Write([]byte(msg))
@@ -27,9 +28,9 @@ func CodeFlow(relyingParty rp.RelyingParty, callbackPath, port string, stateProv
 	http.Handle(loginPath, rp.AuthURLHandler(stateProvider, relyingParty))
 	http.Handle(callbackPath, rp.CodeExchangeHandler(callback, relyingParty))
 
-	utils.StartServer(ctx, port)
+	utils.StartServer(codeflowCtx, ":"+port)
 
 	utils.OpenBrowser("http://localhost:" + port + loginPath)
 
-	return token
+	return <-tokenChan
 }

pkg/utils/http.go

@@ -97,7 +97,11 @@ func StartServer(ctx context.Context, port string) {
 
 	go func() {
 		<-ctx.Done()
-		err := server.Shutdown(ctx)
+		ctxShutdown, cancelShutdown := context.WithTimeout(context.Background(), 5*time.Second)
-		log.Fatalf("Shutdown(): %v", err)
+		defer cancelShutdown()
+		err := server.Shutdown(ctxShutdown)
+		if err != nil {
+			log.Fatalf("Shutdown(): %v", err)
+		}
 	}()
 }