From 8e6632736c194dc4286f402f62b9404b0eba904c Mon Sep 17 00:00:00 2001
From: Charles Lowell <10964656+chlowell@users.noreply.github.com>
Date: Wed, 19 Apr 2023 11:43:18 -0700
Subject: [PATCH] separate assignment from return

---
 apps/internal/oauth/oauth.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/apps/internal/oauth/oauth.go b/apps/internal/oauth/oauth.go
index c3080759..ebd86e2b 100644
--- a/apps/internal/oauth/oauth.go
+++ b/apps/internal/oauth/oauth.go
@@ -113,7 +113,8 @@ func (t *Client) Credential(ctx context.Context, authParams authority.AuthParams
 		tr, err := cred.TokenProvider(ctx, params)
 		if err != nil {
 			if len(scopes) == 0 {
-				return accesstokens.TokenResponse{}, fmt.Errorf("token request had an empty authority.AuthParams.Scopes, which may cause the following error: %w", err)
+				err = fmt.Errorf("token request had an empty authority.AuthParams.Scopes, which may cause the following error: %w", err)
+				return accesstokens.TokenResponse{}, err
 			}
 			return accesstokens.TokenResponse{}, err
 		}
@@ -220,7 +221,8 @@ func (t *Client) UsernamePassword(ctx context.Context, authParams authority.Auth
 		return tr, nil
 	case authority.Managed:
 		if len(authParams.Scopes) == 0 {
-			return accesstokens.TokenResponse{}, fmt.Errorf("token request had an empty authority.AuthParams.Scopes, which may cause the following error: %w", err)
+			err = fmt.Errorf("token request had an empty authority.AuthParams.Scopes, which may cause the following error: %w", err)
+			return accesstokens.TokenResponse{}, err
 		}
 		return t.AccessTokens.FromUsernamePassword(ctx, authParams)
 	}