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

Added new feature/Refactor Authentication System: Added Refresh Token Handling and Cookie Management #37

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mesalm
Copy link

@mesalm mesalm commented Oct 9, 2024

This pull request includes significant updates to the authentication system, focusing on the following changes:
Refresh Token Management:
Implemented the generation and storage of refresh tokens in cookies during user registration and login.
Access Token Refresh Endpoint:
Created an endpoint to handle the refresh token, which validates the token and generates a new access token.
Error Handling Improvements:
Enhanced error handling to provide clearer messages for invalid or expired tokens.
Security Enhancements:
Updated cookie settings to improve security by using HttpOnly and Secure flags.

These changes improve the overall security and user experience by allowing users to maintain their sessions seamlessly. Please review the code modifications and provide feedback.

Copy link

vercel bot commented Oct 9, 2024

@mesalm is attempting to deploy a commit to the sajalbatra's projects Team on Vercel.

A member of the Team first needs to authorize it.

@mesalm
Copy link
Author

mesalm commented Oct 9, 2024

"Hello, I have made some modifications to the Foodzy project, including edits to the header part and the addition of new pages. I would appreciate it if you could review my pull request. Thank you!"

@mesalm
Copy link
Author

mesalm commented Oct 10, 2024

I hope you accept my contribution.

@mesalm
Copy link
Author

mesalm commented Oct 26, 2024

Thank you for the feedback and for adding the hacktoberfest-accepted label to the pull request! I wanted to check if there’s anything else needed on my end for this PR to be merged, or if you had any additional comments before proceeding. Please let me know if I can assist with any final changes.

Looking forward to your response, and thank you for the opportunity to contribute!
@sajalbatra

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