-
Notifications
You must be signed in to change notification settings - Fork 13k
fix: OAuth errors are not handled properly due to an internal exception on the error handling code #35852
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
Conversation
…on on the error handling code
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
🦋 Changeset detectedLatest commit: 31d40a2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 36 packages
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 |
Kody Review CompleteGreat news! 🎉 Keep up the excellent work! 🚀 Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #35852 +/- ##
========================================
Coverage 61.20% 61.21%
========================================
Files 3007 3007
Lines 71472 71472
Branches 16343 16343
========================================
+ Hits 43748 43753 +5
+ Misses 24755 24750 -5
Partials 2969 2969
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
…redentials collection
Kody Review CompleteGreat news! 🎉 Keep up the excellent work! 🚀 Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
…on on the error handling code (#35852)
Proposed changes (including videos or screenshots)
Issue(s)
Closes #35341
CORE-1153
Steps to test or reproduce
Further comments
This pull request addresses an issue in the Rocket.Chat repository where OAuth errors were not being handled properly due to an internal exception in the error handling code. The changes include a patch fix for the '@rocket.chat/meteor' package, as documented in a newly added changeset file. The fix specifically improves error handling and makes credential removal asynchronous in the OAuth flow, as reflected in the modifications to the
loginHandler.tsfile in the 2FA server app. The source branch for these changes isfix/oauth/remove-credentials-with-errors, and the target branch isdevelop.