RocketChat / Rocket.Chat

The communications platform that puts data protection first.
https://rocket.chat/
Other
40.09k stars 10.35k forks source link

RC 5.4.8: Web UI not loading after OAuth login due to "uncaught promises" #29031

Open andreaswolf opened 1 year ago

andreaswolf commented 1 year ago

Description:

I have a custom OAuth implementation. After logging in on the auth provider, I get redirected back to RC and hear the welcome sound, but I only see the three animated dots and do not get a chat interface.

Reloading the chat fixes the problem.

Steps to reproduce:

Not sure if this is reproducible with a plain RocketChat outside our special setup.

  1. Have a RocketChat instance with a custom OAuth provider (PHP-based, using league/oauth2-server as the base)
  2. Open the chat, click the button "Login via "
  3. Log in with the identity provider, get redirected back to the chat
  4. See the chat open, but breaking with the three animated dots
  5. The browser console shows uncaught exceptions in promises:

image

Expected behavior:

Chat opens normally.

Actual behavior:

Three animated dots, in the console I see uncaught exceptions within promises.

I tried tracing the exceptions to their unminified source, and I think it's these two places:

Server Setup Information:

Client Setup Information

Additional context

Relevant logs:

Stacktrace from Chrome for the rocketchat:streamer error:

onStop @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1458
_livedata_nosub @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1366
onMessage @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1366
(anonymous) @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
forEachCallback @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
socket.onmessage @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
r.dispatchEvent @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
k._dispatchMessage @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
k._didMessage @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
o.ws.onmessage @ 63326d8399da4bede896e472cb4d732145c62c84.js?meteor_js_resource=true:1336
andreaswolf commented 10 months ago

Any news on this? Since RC 5.x is apparently not supported anymore and the native apps (at least on Android) nag all users because of this, I in turn get an influx of user requests currently. Rolling out 6.x however is impossible for us because of this very bug.