Skip to content

fix: clean up OAuth tokens after user deactivation#40629

Merged
julio-rocketchat merged 2 commits into
release-8.2.4from
backport-8.2.4-40524
May 21, 2026
Merged

fix: clean up OAuth tokens after user deactivation#40629
julio-rocketchat merged 2 commits into
release-8.2.4from
backport-8.2.4-40524

Conversation

@dionisio-bot
Copy link
Copy Markdown
Contributor

@dionisio-bot dionisio-bot Bot commented May 20, 2026

Backport of #40524

@dionisio-bot dionisio-bot Bot requested review from a team as code owners May 20, 2026 21:06
@dionisio-bot dionisio-bot Bot requested a review from julio-rocketchat May 20, 2026 21:06
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 20, 2026

🦋 Changeset detected

Latest commit: a8794f1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@rocket.chat/model-typings Patch
@rocket.chat/models Patch
@rocket.chat/meteor Patch
@rocket.chat/core-typings Patch
@rocket.chat/rest-typings Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dionisio-bot dionisio-bot Bot added the backport Used to inform backported PR label May 20, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 12 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread apps/meteor/app/api/server/v1/users.ts
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release-8.2.4@763c275). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##             release-8.2.4   #40629   +/-   ##
================================================
  Coverage                 ?   70.47%           
================================================
  Files                    ?     3194           
  Lines                    ?   112882           
  Branches                 ?    20426           
================================================
  Hits                     ?    79551           
  Misses                   ?    31266           
  Partials                 ?     2065           
Flag Coverage Δ
e2e 60.34% <ø> (?)
e2e-api 47.92% <16.66%> (?)
unit 71.36% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dionisio-bot
Copy link
Copy Markdown
Contributor Author

dionisio-bot Bot commented May 21, 2026

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is missing the 'stat: QA assured' label
  • This PR is missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

@julio-rocketchat julio-rocketchat merged commit 776d43c into release-8.2.4 May 21, 2026
130 of 138 checks passed
@julio-rocketchat julio-rocketchat deleted the backport-8.2.4-40524 branch May 21, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Used to inform backported PR community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant