github-starred/better-auth
2
0
Fork 0
mirror of https://github.com/better-auth/better-auth.git synced 2026-05-23 23:52:05 -05:00
Code Issues 2.5k Packages Projects Releases 112 Wiki Activity

[PR #189] [MERGED] fix: OAuth flow errors #3148

New Issue
Closed
opened 2026-03-13 10:41:40 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-03-13 10:41:40 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/189
Author: @Bekacru
Created: 10/16/2024
Status: ✅ Merged
Merged: 10/16/2024
Merged by: @Bekacru

Base: main ← Head: facebook-oauth-fix

📝 Commits (10+)

  • d551ae2 fix: remove codeverifier
  • b1020d1 chore: release v0.4.10-beta.6
  • 8eac7e6 fix: add claims and proper scopes for twitch
  • 4dcb743 chore: release v0.4.10-beta.6
  • cf9042a Merge branch 'twitch-oauth-fix' into facebook-oauth-fix
  • a48a8a1 chore: release v0.4.10-beta.7
  • e9cd762 fix: generic oauth should use the code from qparam
  • a5c737f chore: release v0.4.10-beta.8
  • 72f1822 fix: scope failing to merge
  • 2de5598 chore: release v0.4.10-beta.9

📊 Changes

9 files changed (+71 additions, -22 deletions)

View changed files

📝 demo/nextjs/components/sign-in.tsx (+23 -1)
📝 demo/nextjs/lib/auth.ts (+4 -0)
📝 packages/better-auth/src/api/routes/forget-password.ts (+1 -1)
📝 packages/better-auth/src/api/routes/sign-in.ts (+1 -1)
📝 packages/better-auth/src/oauth2/create-authorization-url.ts (+17 -1)
📝 packages/better-auth/src/oauth2/utils.ts (+5 -1)
📝 packages/better-auth/src/plugins/generic-oauth/index.ts (+2 -1)
📝 packages/better-auth/src/social-providers/facebook.ts (+3 -4)
📝 packages/better-auth/src/social-providers/twitch.ts (+15 -12)

📄 Description

closes #186

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/189 **Author:** [@Bekacru](https://github.com/Bekacru) **Created:** 10/16/2024 **Status:** ✅ Merged **Merged:** 10/16/2024 **Merged by:** [@Bekacru](https://github.com/Bekacru) **Base:** `main` ← **Head:** `facebook-oauth-fix` --- ### 📝 Commits (10+) - [`d551ae2`](https://github.com/better-auth/better-auth/commit/d551ae2db669db5919bfe60c05c66d51356e7a2c) fix: remove codeverifier - [`b1020d1`](https://github.com/better-auth/better-auth/commit/b1020d184feb494318e81275d62302630376a98b) chore: release v0.4.10-beta.6 - [`8eac7e6`](https://github.com/better-auth/better-auth/commit/8eac7e61973f47b71ae7d55613f3457a3f270dcf) fix: add claims and proper scopes for twitch - [`4dcb743`](https://github.com/better-auth/better-auth/commit/4dcb743b1767cc96350a08336315aec751c6be49) chore: release v0.4.10-beta.6 - [`cf9042a`](https://github.com/better-auth/better-auth/commit/cf9042a8a2f53d705dc38d48db0473e1e95b6ba1) Merge branch 'twitch-oauth-fix' into facebook-oauth-fix - [`a48a8a1`](https://github.com/better-auth/better-auth/commit/a48a8a175a7cf7140f96f89c285b8874e3286b96) chore: release v0.4.10-beta.7 - [`e9cd762`](https://github.com/better-auth/better-auth/commit/e9cd762ecb7b217b7a2e54a0ff41d2e5d742e484) fix: generic oauth should use the code from qparam - [`a5c737f`](https://github.com/better-auth/better-auth/commit/a5c737f68ab7b09923da2793b21a7200608cd924) chore: release v0.4.10-beta.8 - [`72f1822`](https://github.com/better-auth/better-auth/commit/72f1822fb5620a7ac7185491f395387fbf28ceab) fix: scope failing to merge - [`2de5598`](https://github.com/better-auth/better-auth/commit/2de5598f576ccbdb2a24d9dae5d8fb037ad0637d) chore: release v0.4.10-beta.9 ### 📊 Changes **9 files changed** (+71 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `demo/nextjs/components/sign-in.tsx` (+23 -1) 📝 `demo/nextjs/lib/auth.ts` (+4 -0) 📝 `packages/better-auth/src/api/routes/forget-password.ts` (+1 -1) 📝 `packages/better-auth/src/api/routes/sign-in.ts` (+1 -1) 📝 `packages/better-auth/src/oauth2/create-authorization-url.ts` (+17 -1) 📝 `packages/better-auth/src/oauth2/utils.ts` (+5 -1) 📝 `packages/better-auth/src/plugins/generic-oauth/index.ts` (+2 -1) 📝 `packages/better-auth/src/social-providers/facebook.ts` (+3 -4) 📝 `packages/better-auth/src/social-providers/twitch.ts` (+15 -12) </details> ### 📄 Description closes #186 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-03-13 10:41:40 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
adapter
astro
awaiting external contributor
blocked
breaking
breaking change
bug
c-devops
core
credentials
database
dependencies
devops
devtools
docs
documentation
duplicate
elysia
enhancement
enterprise
expo
express
fastify
good first issue
help wanted
hono
identity
infra
integration
invalid
javascript
locked
maintenance
need-more-information
needs: info
needs: repro
nextjs
nuxt
oauth
organization
P0
payments
perf
platform
plugin
pull-request
Mirrored from GitHub Pull Request
 question
ready
regression
remix
security
social-provider
solid
stale
svelte
tanstack-start
tracking
version-bump
vue
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#3148
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?