Skip to content

Commit

Permalink
Fix several issues encountered in the Slack provider (#1147)
Browse files Browse the repository at this point in the history
Co-authored-by: pilcrowOnPaper <[email protected]>
  • Loading branch information
ollema and pilcrowonpaper authored Sep 23, 2023
1 parent 06fbc88 commit 5958f48
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
6 changes: 6 additions & 0 deletions .auri/$saog9jyl.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
package: "@lucia-auth/oauth"
type: "minor"
---

Fix `slack()` provider
5 changes: 3 additions & 2 deletions packages/oauth/src/providers/slack.ts
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ export class SlackAuth<_Auth extends Auth = Auth> extends OAuth2ProviderAuth<
"https://slack.com/openid/connect/authorize",
{
clientId: this.config.clientId,
scope: ["oidc", "profile", ...scopeConfig],
scope: ["openid", "profile", ...scopeConfig],
redirectUri: this.config.redirectUri
}
);
Expand Down Expand Up @@ -79,7 +79,8 @@ export class SlackAuth<_Auth extends Auth = Auth> extends OAuth2ProviderAuth<
clientPassword: {
clientSecret: this.config.clientSecret,
authenticateWith: "client_secret"
}
},
redirectUri: this.config.redirectUri
});
return {
accessToken: tokens.access_token,
Expand Down

0 comments on commit 5958f48

Please sign in to comment.