Apply suggestions from code review

Co-authored-by: Jan Larwig <[email protected]>
oauth2-proxy · Jan 3, 2024 · cd5fbef · cd5fbef
1 parent 92e1fca
commit cd5fbef
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/providers/srht.go b/providers/srht.go
@@ -78,21 +78,21 @@ func (p *SourceHutProvider) EnrichSession(ctx context.Context, s *sessions.Sessi
 		WithMethod("POST").
 		SetHeader("Content-Type", "application/json").
 		SetHeader("Authorization", "Bearer "+s.AccessToken).
-		WithBody(strings.NewReader("{\"query\": \"{ me { username, email } }\"}")).
+		WithBody(bytes.NewBufferString(`{"query": "{ me { username, email } }"}`)).
 		Do().
 		UnmarshalSimpleJSON()
 	if err != nil {
 		logger.Errorf("failed making request %v", err)
 		return err
 	}
 
-	email, err := json.Get("data").Get("me").Get("email").String()
+	email, err := json.GetPath("data", "me", "email").String()
 	if err != nil {
 		return fmt.Errorf("unable to extract email from userinfo endpoint: %v", err)
 	}
 	s.Email = email
 
-	username, err := json.Get("data").Get("me").Get("username").String()
+	username, err := json.GetPath("data", "me", "username").String()
 	if err == nil {
 		s.PreferredUsername = username
 		s.User = username