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

throwOnError mode? #1118

New Issue
Closed
opened 2026-03-13 08:23:30 -05:00 by GiteaMirror · 2 comments
GiteaMirror commented 2026-03-13 08:23:30 -05:00
Owner
Copy Link

Originally created by @mmkal on GitHub (Apr 26, 2025).

Is this suited for github?

  • Yes, this is suited for github

Is your feature request related to a problem? Please describe.

Minor problem - it's cumbersome constantly doing

const {error, data} = await authClient.whatever.foo()
if (error) throw error
return data

Especially so when you have linter rules insisting you always throw Error instances, and when you have multiple calls so you have to rename error and data to disambiguate.

Describe the solution you'd like

A new prop you can pass into to createAuthClient which changes the return type and behaviour of the various async methods:

export const authClient = createAuthClient({
  // other options
  throwOnErrror: true,
})

Describe alternatives you've considered

Writing a wrapper for createAuthClient myself - but the types get pretty complicated and are dependent on specifics of better-auth.

Additional context

This kind of API would also be more suited to libraries like react-query which keep track of errors out of the box.

Originally created by @mmkal on GitHub (Apr 26, 2025). ### Is this suited for github? - [x] Yes, this is suited for github ### Is your feature request related to a problem? Please describe. Minor problem - it's cumbersome constantly doing ```ts const {error, data} = await authClient.whatever.foo() if (error) throw error return data ``` Especially so when you have linter rules insisting you always throw `Error` instances, and when you have multiple calls so you have to rename `error` and `data` to disambiguate. ### Describe the solution you'd like A new prop you can pass into to `createAuthClient` which changes the return type and behaviour of the various async methods: ```ts export const authClient = createAuthClient({ // other options throwOnErrror: true, }) ``` ### Describe alternatives you've considered Writing a wrapper for `createAuthClient` myself - but the types get pretty complicated and are dependent on specifics of better-auth. ### Additional context This kind of API would also be more suited to libraries like react-query which keep track of errors out of the box.
GiteaMirror commented 2026-03-13 08:23:31 -05:00
Author
Owner
Copy Link

@Bekacru commented on GitHub (Apr 28, 2025):

this is already available

export const authClient = createAuthClient({
  fetchOptions: {
    throw: true
  }
})
@Bekacru commented on GitHub (Apr 28, 2025): this is already available ```ts export const authClient = createAuthClient({ fetchOptions: { throw: true } }) ```
GiteaMirror commented 2026-03-13 08:23:35 -05:00
Author
Owner
Copy Link

@mmkal commented on GitHub (May 16, 2025):

Sorry missed the reply - thank you !

@mmkal commented on GitHub (May 16, 2025): Sorry missed the reply - thank you !
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
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#1118
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?