Fix renamed octokit/rest listEmails call (#127)

purplexa · web-flow · commit f434637fa2ac · 2022-01-24T13:43:45.000-08:00
diff --git a/api_server/modules/github_auth/github_authorizer.js b/api_server/modules/github_auth/github_authorizer.js
@@ -20,7 +20,7 @@ class GithubAuthorizer {
 		this.providerInfo = this.providerInfo || providerInfo;
 		this.octokit = new Octokit({ auth: accessToken });
 		const userData = await this.githubApiRequest('users', 'getAuthenticated');
-		let emailData = await this.githubApiRequest('users', 'listEmails');
+		let emailData = await this.githubApiRequest('users', 'listEmailsForAuthenticatedUser');
 		this.request.log('user data: ' + JSON.stringify(userData, undefined, 5));
 		this.request.log('email data: ' + JSON.stringify(emailData, undefined, 5));
 		emailData = emailData instanceof Array ? emailData : null;
@@ -53,7 +53,7 @@ class GithubAuthorizer {
 			if (method === 'getAuthenticated') {
 				return this._mockIdentity(mockCode[1]);
 			}
-			else if (method === 'listEmails') {
+			else if (method === 'listEmailsForAuthenticatedUser') {
 				return this._mockEmails();
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ class GithubAuthorizer {`
`20`	`20`	`this.providerInfo = this.providerInfo \|\| providerInfo;`
`21`	`21`	`this.octokit = new Octokit({ auth: accessToken });`
`22`	`22`	`const userData = await this.githubApiRequest('users', 'getAuthenticated');`
`23`		`- let emailData = await this.githubApiRequest('users', 'listEmails');`
	`23`	`+ let emailData = await this.githubApiRequest('users', 'listEmailsForAuthenticatedUser');`
`24`	`24`	`this.request.log('user data: ' + JSON.stringify(userData, undefined, 5));`
`25`	`25`	`this.request.log('email data: ' + JSON.stringify(emailData, undefined, 5));`
`26`	`26`	`emailData = emailData instanceof Array ? emailData : null;`
`@@ -53,7 +53,7 @@ class GithubAuthorizer {`
`53`	`53`	`if (method === 'getAuthenticated') {`
`54`	`54`	`return this._mockIdentity(mockCode[1]);`
`55`	`55`	`}`
`56`		`- else if (method === 'listEmails') {`
	`56`	`+ else if (method === 'listEmailsForAuthenticatedUser') {`
`57`	`57`	`return this._mockEmails();`
`58`	`58`	`}`
`59`	`59`	`}`