Skip to content

Implement logout endpoint on API#285

Merged
Xhristin3 merged 2 commits into
XStreamRollz:mainfrom
sadeeq6400:feat/logOutAPI
Jun 17, 2026
Merged

Implement logout endpoint on API#285
Xhristin3 merged 2 commits into
XStreamRollz:mainfrom
sadeeq6400:feat/logOutAPI

Conversation

@sadeeq6400

Copy link
Copy Markdown
Contributor

Summary

This PR has fullfilled all the requirements needed

Provide a short description of the changes in this PR and the motivation/why.

Related issues

Link any related issues. Use Closes #<id> to close issues automatically.

Closes: #198

Type of change

  • [ x ] feat
  • fix
  • docs
  • chore
  • test
  • ci

Match the PR title to Conventional Commits: <type>(<scope>): <summary>.

Testing performed

Describe how you tested this change. Include commands, environment, and any setup steps.

Example:

cd api
npm run lint
npm run build
npm test

Screenshots (if applicable)

Attach screenshots or animated GIFs for UI changes.

Checklist — author

  • I rebased onto origin/main and resolved conflicts.
  • I ran the quality gates locally: npm run lint, npm run build, npm test.
  • Title follows Conventional Commits and references the issue (see above).
  • I added/updated tests where applicable and they pass locally.
  • No new TypeScript errors or lint warnings introduced.
  • I updated documentation if the change affects public behavior.
  • Screenshots included for UI changes.

Checklist — reviewer guidance

  • Required checks: lint, build, unit tests, and any package-specific E2E must be green.
  • Request review from CODEOWNER(s) for touched packages.
  • Prefer small, focused PRs. If large, confirm feature-flagging or follow-up tasks.

Follow the full PR process in CONTRIBUTING.md — particularly: sync with main, run quality gates, request a CODEOWNER review, and use squash-and-merge with a Conventional Commit-style squash message.

@Xhristin3 Xhristin3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Token denylist tied to remaining TTL + guard check on every request is the right shape. Replacing the placeholder X-User-Id header with real JWT validation is a great cleanup too. Tests look good. Merging 🎉

@Xhristin3 Xhristin3 merged commit 876e112 into XStreamRollz:main Jun 17, 2026
2 checks passed
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.

feat: Implement logout endpoint on API

2 participants