From 514f6ec10c42a51eadf39021795bddf9d1cdec7d Mon Sep 17 00:00:00 2001 From: max furman Date: Thu, 16 Jun 2022 14:28:46 -0700 Subject: [PATCH] Simply a few variable declarations in conditionals --- command/oauth/cmd.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/command/oauth/cmd.go b/command/oauth/cmd.go index d8e3d4f3..c5eeca1d 100644 --- a/command/oauth/cmd.go +++ b/command/oauth/cmd.go @@ -579,15 +579,17 @@ func newOauth(provider, clientID, clientSecret, authzEp, deviceAuthzEp, tokenEp, if _, ok := d["device_authorization_endpoint"]; !ok && opts.Device { return nil, errors.New("missing 'device_authorization_endpoint' in provider metadata") + } else if ok { + deviceAuthzEp = d["device_authorization_endpoint"].(string) } if _, ok := d["authorization_endpoint"]; !ok && !opts.Device { return nil, errors.New("missing 'authorization_endpoint' in provider metadata") + } else if ok { + authzEp = d["authorization_endpoint"].(string) } if _, ok := d["token_endpoint"]; !ok { return nil, errors.New("missing 'token_endpoint' in provider metadata") } - deviceAuthzEp = d["device_authorization_endpoint"].(string) - authzEp = d["authorization_endpoint"].(string) tokenEp = d["token_endpoint"].(string) userinfoEp = d["token_endpoint"].(string) }