Authentication! Verification! Frustration!#19
Merged
AlexVOiceover merged 25 commits intomainfrom Apr 1, 2025
Merged
Conversation
- Create detailed task plan for auth implementation
- Document token, session management, and email procedures
- Outline step-by-step implementation approach for all endpoints
- Align implementation plan with frontend requirements
- Replaced custom JWT implementation with better-auth library - Updated task plan to directly integrate with frontend's library - Added better-auth dependency to deno.jsonc 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add better-auth dependency - Create authConfig with magic link plugin - Create Deno KV user store for auth data - Implement magic link email sending functionality
- Create authConfig with magic link plugin - Create Deno KV user store for auth data - Implement magic link email sending functionality
- Create unit tests for DenoKvUserStore with in-memory implementation - Add tests for magic link email functionality with fetch mocking - Implement auth configuration tests with proper environment setup - Update test commands with necessary unstable flags and permissions - Add auth integration test for end-to-end validation
- Create placeholder auth handlers in authConfig.ts - Update auth routes with proper request/response handling - Add validation for required parameters in auth endpoints - Return mock user data for development testing
- Create placeholder auth handlers until better-auth is properly integrated - Implement proper request/response handling in auth routes - Add validation for email and token parameters - Return mock user data for development testing
- Replace placeholder auth object with proper better-auth initialization - Update auth routes to use better-auth APIs for magic link flow - Add detailed console logging throughout auth process - Create test endpoint to verify auth configuration - Update authMiddleware to work with better-auth session management - Improve error handling and response formats 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix import paths for better compatibility
- Add detailed debugging to determine auth object structure
- Implement fallback methods for magic link functionality
- Make test endpoint report actual auth structure
- Add better error handling throughout auth routes
- Complete better-auth integration using the handler approach
- Add diagnostic logging to debug auth structure
- Ensure routes properly use the handler for auth operations
- Provide development fallbacks for easier testing
- Improve error handling and response formatting
🚀 PR Summary📝 Changes Overview✨ Authentication! Verification! Frustration! 📂 Affected Files
😄 Code HumorWhy did the programmer quit their job? They didn't get arrays. 👨💻 This summary was automatically generated by PR Summarizer ⚡ |
AlexVOiceover
approved these changes
Apr 1, 2025
Contributor
AlexVOiceover
left a comment
AlexVOiceover
left a comment
There was a problem hiding this comment.
Yeah, lots of staff here, right?
I am happy to scrap better-auth and use Supabase instead. And it only took 915 new lines :)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ALL OF THIS, IN THE BIN, STRAIGHT AWAY, GET IN
What DOESN'T this update do?
So sit back, relax and enjoy ELEVEN DAYS OF CODING in one big, totally unreviewable PR.
BUT WAIT, THERE'S
LESSMORE!Comes bundled with the free "Library of Despair", a set of commits that acts as a realtime chronicle of one man's journey through recursive function hell. Why not start your journey with the commit where I...
IN SUMMARY
Please enact the following action points on the following packages
if (bin) { bin.push(pg) };