Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add spotify oauth adapter #73

Merged
merged 3 commits into from
Dec 14, 2024
Merged

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Add Spotify OAuth adapter implementing authorization code flow.

  • Implements standard OAuth2 authorization code flow with authorization code grant type
  • Supports essential scopes:
    • user-read-private (basic profile)
    • user-read-email (email access)
    • playlist-read-private (playlist access)
    • user-library-read (saved content)
  • Follows existing adapter pattern used by other OAuth providers
  • Configures Spotify-specific OAuth endpoints:

Link to Devin run: https://app.devin.ai/sessions/b8acf68438ab49fd86868b5d2331d63f

Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Add "(aside)" to your comment to have me ignore it.

Copy link

changeset-bot bot commented Dec 14, 2024

🦋 Changeset detected

Latest commit: c1614b8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@openauthjs/openauth Patch
nextjs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

authorization: "https://accounts.spotify.com/authorize",
token: "https://accounts.spotify.com/api/token",
},
scopes: [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't specify the scopes let the user pass through

@thdxr thdxr merged commit 83125f1 into master Dec 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant