Skip to content

Commit e31dc83

Browse files
committed
Look at the user login instead of the user id for clarity
1 parent 7cd15d5 commit e31dc83

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

lib/github-oauth.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -206,13 +206,13 @@ export function registerGitHubOAuth(server: FastifyInstance, config: Config) {
206206
const user = await getGitHubUser(tokenData)
207207
const members = await getGitHubOrgMemberships()
208208

209-
console.log("User: ", user.id)
209+
console.log("User: ", user.login)
210210
console.log(
211211
"Members: ",
212-
members.map((member) => member.id),
212+
members.map((member) => member.login),
213213
)
214214

215-
if (!members.find((member) => member.id === user.id)) {
215+
if (!members.find((member) => member.login === user.login)) {
216216
return res.redirect(302, urls.localMembershipError)
217217
}
218218

0 commit comments

Comments
 (0)