Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented /api/session Endpoint #37

Merged
merged 8 commits into from
Jul 16, 2023
Merged

Implemented /api/session Endpoint #37

merged 8 commits into from
Jul 16, 2023

Conversation

bliutech
Copy link
Owner

Summary

Closes #20. Adds user sessions API for authentication and adds authentication middleware. This pull request is meant not only to implement this feature but also to demonstrate incremental changes for development by breaking up feature development into multiple commits.

Test Plan

Integrate into existing endpoints. 👍

bliutech and others added 8 commits July 16, 2023 00:26
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
account for edge case where users exist but some have been deleted when restarting the application

Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Co-authored-by: Trisha Reddy <105570654+trishavreddy@users.noreply.github.com>
Co-authored-by: Chloe <138538887+ChL4859@users.noreply.github.com>
Copy link
Collaborator

@cy-ros cy-ros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cy-ros cy-ros merged commit 25b1c79 into main Jul 16, 2023
@cy-ros cy-ros deleted the session-api branch July 16, 2023 22:52
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.

🚀 Feature[backend]: Implement /api/session Endpoint
2 participants