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

Fix logout API validation messages #104

Closed
krschan opened this issue Mar 28, 2025 · 1 comment · Fixed by #108
Closed

Fix logout API validation messages #104

krschan opened this issue Mar 28, 2025 · 1 comment · Fixed by #108
Assignees
Labels
⬅️BACKEND 🐛bug Something isn't working

Comments

@krschan
Copy link
Collaborator

krschan commented Mar 28, 2025

Fixes from PullRequest: #105

  1. Fix Validation Message for Reusing Token

    • Ensure the validation message is accurate when the user attempts to log out using the same token in Postman.
    • Modify the response to give a more specific error message when the same token is reused.
  2. Update Response When No Token is Provided

    • Modify the response when the user doesn't provide a token in Postman for the logout API.
    • The error message should be similar to the one from the register-step-two API.
  3. Test API Response

    • Write test cases to verify that the validation message is accurate when the user attempts to log out without providing a token.
    • Ensure that the error message is consistent with the expected format for missing token scenarios.
@krschan krschan moved this to Backlog in COPAY - Sprint 1 Mar 28, 2025
@krschan krschan added 🐛bug Something isn't working ⬅️BACKEND labels Mar 28, 2025
@poksyy poksyy self-assigned this Mar 30, 2025
@poksyy
Copy link
Owner

poksyy commented Mar 30, 2025

@krschan The second part of the issue will have to be created because it is more complex and not directly related to the first part.

Backend messages for token validation will need to be improved.

CustomAuthenticationEntryPoint.java and valideToken method on JwtService.java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⬅️BACKEND 🐛bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants