Skip to content

Feat/tanstack demo#55

Open
mrhut10 wants to merge 9 commits intozpg6:mainfrom
mrhut10:feat/tanstack-demo
Open

Feat/tanstack demo#55
mrhut10 wants to merge 9 commits intozpg6:mainfrom
mrhut10:feat/tanstack-demo

Conversation

@mrhut10
Copy link
Copy Markdown

@mrhut10 mrhut10 commented Apr 23, 2026

  • assumes that support for configs in jsonc or json #54 has merged as documentation mentions updating your .jsonc config file
  • have initially deployed the demo to https://better-auth-cloudflare-tanstack-template.ahut10.workers.dev/ however please feel free to redeploy under a project controlled cloudflare account
  • for some reason the getRequest method by tanstack-start isn't populating the request.cf which means the clients locale/geo information isn't coming though.
    • however when I manually injected a log in the workers fetch handler obsolutely CF is providing the request.CF so this may be a current bug in tanstack libary itself.

I'm still learning the better-auth eco system, and deploying to cloudflare, and contributing to this project so please point me in the right direction if could be done better.

@mrhut10
Copy link
Copy Markdown
Author

mrhut10 commented Apr 23, 2026

have just added a commit to fix my local resolution to a branch that had the #54 in it so it now should be ready for a review.

please note @zpg6 my documentation does assume you've already got this MR merged but can be updated to tell user to update the .toml file if need be as hasn't been merged yet

@mrhut10 mrhut10 marked this pull request as ready for review April 23, 2026 00:40
@mrhut10 mrhut10 changed the title draft: Feat/tanstack demo Feat/tanstack demo Apr 23, 2026
@zpg6
Copy link
Copy Markdown
Owner

zpg6 commented Apr 23, 2026

@mrhut10 Nice! Will take a look and provide feedback

@mrhut10
Copy link
Copy Markdown
Author

mrhut10 commented Apr 23, 2026

just redeployed to fix the schema in D1 in the example, signup/signin is working now.

@mrhut10
Copy link
Copy Markdown
Author

mrhut10 commented Apr 24, 2026

Just realised @zpg6 the deployed example most of the time on sign-up/signin the Hashing func is exceeding CPU usage of my free cloudflare account ( 10ms limit 🤦)

Next week I'll upgrade to a paid account but you may have to deploy your own copy to see it working end to end before next week apologies

effected by #56

I'll upgrade my plan next week which should mitigate and I'll look to alter the hashing settings to what I deem safer as well.

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.

2 participants