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

feat: create refresh token #1327

Closed
wants to merge 2 commits into from

Conversation

theChosenDevop
Copy link

Pull Request

Description

This PR implements the following authentication-related features:

  • Refresh Token Handling: Generates and stores HTTP-only refresh tokens for improved security.
  • Logout Functionality: Ensures secure logout by clearing the refresh token from cookies.
  • Login Enhancements: Adds improved token generation and user validation.

Related Issue

Fixes # 1260

Type of Change

  • feat: New feature
  • fix: Bug fix
  • docs: Documentation updates
  • style: Code style/formatting changes
  • refactor: Code refactoring
  • perf: Performance improvements
  • test: Test additions/updates

Test Evidence

All tests passed successfully, including:

  • Login with valid credentials
  • Login with invalid credentials
  • Refresh token issuance and validation
  • Logout functionality ensuring refresh token is removed

Screenshots (if applicable)

N/A

Documentation Screenshots (if applicable)

  • Updated API documentation for POST /auth/login, POST /auth/refresh-token, and POST /auth/logout.

Checklist

  • My code follows the project's coding style
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@theChosenDevop theChosenDevop deleted the newfeature branch March 1, 2025 15:11
@theChosenDevop theChosenDevop restored the newfeature branch March 1, 2025 15:11
@theChosenDevop theChosenDevop deleted the newfeature branch March 1, 2025 15:13
@theChosenDevop theChosenDevop restored the newfeature branch March 1, 2025 15:41
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.

1 participant