Add auth service for signup, login, and token handling#166
Add auth service for signup, login, and token handling#166harsh-hak wants to merge 6 commits intoGopher-Industries:masterfrom
Conversation
… and token management, and add initial health news tests.
…and token management, and refactor server to be exportable.
|
Hey Harsh , thanks for the update. I can’t approve this PR yet because it currently has a merge conflict in server.js (CORS section). Fix suggestion: when resolving, please combine the two CORS approaches: keep the whitelist callback logic from master (so Postman/curl + localhost variants still work), but also include FRONTEND_ORIGIN as the primary allowed origin. Also small improvement: can we avoid hardcoding FRONTEND_ORIGIN = "http://localhost:3000" and instead use process.env.FRONTEND_ORIGIN || "http://localhost:3000"? Once the conflict is resolved and pushed (and CORS is consistent), tag me again , I’ll re-review and approve. Thanks |
|
Hi Harsh, CI is currently failing because the workflow expects To avoid breaking the pipeline for this multi-trimester project, please restore/revert the deletion of Thanks |
Adds user authentication service supporting signup, login, and secure token handling. Refactors server to allow clean exports and easier testing.