-
Notifications
You must be signed in to change notification settings - Fork 453
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
Chat loads forever sometimes #9493
Comments
I have already (I think) fixed the issue locally, opening the file SystemMessage.php I modified the failing function, simply adding "". before the $i to ensure it's a string when looking up in the paramaters['users'][RIGHT_HERE] and the chat loads again. |
Did you by chance had a call with a user that only has numbers as user id? |
We did manage a video call, that when ended says @username ended the call with {user1} (Duration 1:29) in this, both user id's should be only numbers as assigned by authentik in my setup |
I managed to create the problem in a test |
In case you want to test the patch: |
Reverted back to original, applied your patch, confirming the chat is still loading. Thank you! |
How to use GitHub
Steps to reproduce
behind Authentik2. Use authentik's user id # for nextcloud's UID * I no longer believe this has anything to do with it, it just happened to be at a function call that gets the user id so I thought maybe it has something to do with itExpected behaviour
Old chats should load
Actual behaviour
One chat I have with one user only loads up around 1/50 times refreshing the page. It doesn't load on the mobile app either. The other conversations I'm having work fine. the only clue i have is the 500 server error and the nextcloud.log log is related to the
UID being interpreted as int's(loop variable $i is int but this version of? php expects str for index into (this?)map)instead of strings but they're required to be strings, also is strange(to me! Probably not to the program) that the log shows "no app in context" instead of "spreed". The messages clearly send since the "last message" (right under the other users username) does always show the last thing sent in that conversation, even though the conversation itself will never load.In this conversation we had exchanged text messages, tested Video chat (Pre- signaling server but post installing and configuring coturn) that worked well, I uploaded a screenshot from my android phone using the talk app to the conversation, and was switching between my PC and my phone with the app frequently.
Talk app
**Talk app version: 16.0.3
Custom Signaling server configured: yes / github main branch from yesterday
Custom TURN server configured: yes - coturn:latest docker image
Custom STUN server configured: yes
Browser
Microphone available: yes
Camera available: yes
Operating system: Windows/Android
Browser name: Firefox/Chrome/Android Talk App...
Browser version: 85/96/...
Browser log
Server configuration
Operating system: Ubuntu 22.04
Web server: OpenLiteSpeed
Database: PostgreSQL
PHP version: 8.1.2-1ubuntu2.11
Nextcloud Version: Nextcloud Hub 4 (26.0.1)
List of activated apps:
activity cloud_federation_api dav files files_sharing logreader oauth2 provisioning_api settings suspicious_login twofactor_backupcodes user_status
admin_audit comments encryption files_external files_trashbin lookup_server_connector password_policy recommendations sharebymail systemtags twofactor_totp viewer
bruteforcesettings contactsinteraction federatedfilesharing files_pdfviewer files_versions nextcloud_announcements photos related_resources support text updatenotification weather_status
circles dashboard federation files_rightclick firstrunwizard notifications privacy serverinfo survey_client theming user_ldap workflowengine
Nextcloud configuration:
Server log (data/nextcloud.log)
The text was updated successfully, but these errors were encountered: