From 9b519758c4ccd4af99fdbb26c85283c0d80e4039 Mon Sep 17 00:00:00 2001 From: The Magician Date: Wed, 16 Nov 2022 12:58:00 -0800 Subject: [PATCH] Fix provider crash when userinfo email is nil (#6839) (#4892) Fixes https://github.com/hashicorp/terraform-provider-google/issues/13051 Signed-off-by: Modular Magician Signed-off-by: Modular Magician --- .changelog/6839.txt | 3 +++ google-beta/utils.go | 3 +++ 2 files changed, 6 insertions(+) create mode 100644 .changelog/6839.txt diff --git a/.changelog/6839.txt b/.changelog/6839.txt new file mode 100644 index 0000000000..b51bc5b20c --- /dev/null +++ b/.changelog/6839.txt @@ -0,0 +1,3 @@ +```release-note:bug +provider: fixed a crash during provider authentication for certain environments +``` diff --git a/google-beta/utils.go b/google-beta/utils.go index 1b932c4ca8..4ff797235c 100644 --- a/google-beta/utils.go +++ b/google-beta/utils.go @@ -508,6 +508,9 @@ func GetCurrentUserEmail(config *Config, userAgent string) (string, error) { if err != nil { return "", fmt.Errorf("error retrieving userinfo for your provider credentials. have you enabled the 'https://www.googleapis.com/auth/userinfo.email' scope? error: %s", err) } + if res["email"] == nil { + return "", fmt.Errorf("error retrieving email from userinfo. email was nil in the response.") + } return res["email"].(string), nil }