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

Cookie is not being set if defaultCookieAttributes path starts with // #784

New Issue
Closed
opened 2026-03-13 08:04:12 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-03-13 08:04:12 -05:00
Owner
Copy Link

Originally created by @krishna-santosh on GitHub (Mar 5, 2025).

Is this suited for github?

  • Yes, this is suited for github

To Reproduce

set defaultCookieAttributes path to '//' in betterAuth config

Current vs. Expected behavior

Currently it does nothing, and browser ignores path as it is not a valid value for the cookie's path attribute. Resulting in no cookie being set.

Expected:
Log an Error

ERROR [Better Auth]: Invalid defaultCookieAttributes path: //

This helps avoid wasting time on debugging when user makes a mistake.

Something like this is already handled appropriately when callbackURL starts with //.

What version of Better Auth are you using?

1.2.3

Provide environment information

NA

Which area(s) are affected? (Select all that apply)

Backend

Auth config (if applicable)

export const auth = betterAuth({
  database: prismaAdapter(prisma, { provider: 'postgresql' }),
  socialProviders: {
    google: {
      clientId: process.env.AUTH_GOOGLE_ID,
      clientSecret: process.env.AUTH_GOOGLE_SECRET,
    },
  },
  advanced: {
    defaultCookieAttributes: {
      path: '//',
    },
  },
});

Additional context

No response

Originally created by @krishna-santosh on GitHub (Mar 5, 2025). ### Is this suited for github? - [x] Yes, this is suited for github ### To Reproduce set `defaultCookieAttributes` path to '//' in betterAuth config ### Current vs. Expected behavior Currently it does nothing, and browser ignores path as it is not a valid value for the cookie's path attribute. Resulting in no cookie being set. Expected: Log an Error > ERROR [Better Auth]: Invalid defaultCookieAttributes path: // This helps avoid wasting time on debugging when user makes a mistake. Something like this is already handled appropriately when callbackURL starts with //. ### What version of Better Auth are you using? 1.2.3 ### Provide environment information ```bash NA ``` ### Which area(s) are affected? (Select all that apply) Backend ### Auth config (if applicable) ```typescript export const auth = betterAuth({ database: prismaAdapter(prisma, { provider: 'postgresql' }), socialProviders: { google: { clientId: process.env.AUTH_GOOGLE_ID, clientSecret: process.env.AUTH_GOOGLE_SECRET, }, }, advanced: { defaultCookieAttributes: { path: '//', }, }, }); ``` ### Additional context _No response_
GiteaMirror added the bug label 2026-03-13 08:04:12 -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 bug
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#784
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?