Must have struct for json to work, not nil (#70)

yorinasub17 · web-flow · commit 3f0ebb82bd9d · 2022-07-14T13:06:18.000-05:00
diff --git a/github/auth.go b/github/auth.go
@@ -62,7 +62,7 @@ func LoadGithubAppConfigFromEnv(envVarName string) (*GithubAppConfig, error) {
 		return nil, err
 	}
 
-	var githubAppConfig *GithubAppConfig
-	jsonLoadErr := json.Unmarshal([]byte(githubAppConfigJSON), githubAppConfig)
-	return githubAppConfig, errors.WithStackTrace(jsonLoadErr)
+	var githubAppConfig GithubAppConfig
+	jsonLoadErr := json.Unmarshal([]byte(githubAppConfigJSON), &githubAppConfig)
+	return &githubAppConfig, errors.WithStackTrace(jsonLoadErr)
 }

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ func LoadGithubAppConfigFromEnv(envVarName string) (*GithubAppConfig, error) {`
`62`	`62`	`return nil, err`
`63`	`63`	`}`
`64`	`64`
`65`		`- var githubAppConfig *GithubAppConfig`
`66`		`- jsonLoadErr := json.Unmarshal([]byte(githubAppConfigJSON), githubAppConfig)`
`67`		`- return githubAppConfig, errors.WithStackTrace(jsonLoadErr)`
	`65`	`+ var githubAppConfig GithubAppConfig`
	`66`	`+ jsonLoadErr := json.Unmarshal([]byte(githubAppConfigJSON), &githubAppConfig)`
	`67`	`+ return &githubAppConfig, errors.WithStackTrace(jsonLoadErr)`
`68`	`68`	`}`