search

prasenjeet-symon / intellectia

Intellectia - Your Guardian Against Misleading AI-Generated Content
MIT License
15 stars 43 forks source link

Logout all route tasks #246

Closed Syphonphilter closed 10 months ago

Syphonphilter commented 10 months ago

This PR is #242 Pull Request: Enhance "/logout_all" Route Refactoring

Summary: This pull request aims to improve the "/logout_all" route located in packages/server/src/api_auth/index.ts. The main objective is to enhance the code structure by using res.locals.reqClientData as the main source for client data instead of direct accesses to req.body, req.query, and req.params. Additionally, it introduces a robust try-catch block for effective error handling, aligning with the established methodology seen in the "login" route within the same directory.

Changes Implemented:

The following changes have been made:

  1. Instead of directly accessing req.body, req.query, and req.params, they are now being accessed through res.locals.reqClientData via the refactored IRequestAuthLogoutAll Interface.
  2. The way client data is extracted from res.locals.reqClientData has been made consistent throughout the codebase.
  3. A try-catch block has been added to handle errors in a more graceful manner.
  4. Error messages and responses have been customized to provide more informative feedback.
  5. The updated "/logout_all" route has been thoroughly tested to ensure smooth functionality.
  6. No changes are needed in the documentation.

Testing Details:

  1. Thoroughly tested the refactored "/logout_all" route to confirm its functionality.
  2. Verified the effectiveness of error handling across diverse scenarios.