-
Notifications
You must be signed in to change notification settings - Fork 437
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
spreed hangs with http2 #225
Comments
No idea what could cause this. |
So how do we continue here? @nickvergessen is more info needed? @JSoko does this also happen on other devices? |
With spreed 1.1.0 it worked without problems.
Yes. I tested 1.2.0 on 2 Notebooks and 3 computers. All 5 have that issue with firefox and chrome. Inside and outside my network under Win10 and sbs8. |
Test Account would be awesome |
@nickvergessen You have an account... |
Okay, seems like the signaling request never ends. I don't think we changed something about this. I can also still not reproduce this locally. Are you using php-fpm or something similar? |
Yes, you have to close the browser...
No. Server is a odroid xu4. 32bit ubuntu 16.04 lts System Linux OdNc 3.10.103 #1 SMP PREEMPT Sat Sep 10 14:43:59 UTC 2016 armv7l Apache Version Apache/2.4.25 (Ubuntu) Apache Environment HTTP Headers Information apcu Core ctype $_SERVER['SERVER_SOFTWARE'] Apache/2.4.25 (Ubuntu) $_SERVER['SERVER_NAME'] of99.net $_SERVER['SERVER_ADDR'] 192.168.1.234 $_SERVER['SERVER_PORT'] 443 $_SERVER['REMOTE_ADDR'] 192.168.1.18 $_SERVER['DOCUMENT_ROOT'] /var/www/cloud $_SERVER['REQUEST_SCHEME'] https $_SERVER['CONTEXT_PREFIX'] /test $_SERVER['CONTEXT_DOCUMENT_ROOT'] /var/www/test/ $_SERVER['SERVER_ADMIN'] [email protected] $_SERVER['SCRIPT_FILENAME'] /var/www/test/phptst.php $_SERVER['REMOTE_PORT'] 62148 $_SERVER['GATEWAY_INTERFACE'] CGI/1.1 $_SERVER['SERVER_PROTOCOL'] HTTP/2.0 $_SERVER['REQUEST_METHOD'] GET $_SERVER['QUERY_STRING'] no value $_SERVER['REQUEST_URI'] /test/phptst.php $_SERVER['SCRIPT_NAME'] /test/phptst.php $_SERVER['PHP_SELF'] /test/phptst.php $_SERVER['REQUEST_TIME_FLOAT'] 1485169037.794 $_SERVER['REQUEST_TIME'] 1485169037 |
Can you check whether |
Nothing related: [Mon Jan 23 06:25:05.421627 2017] [mpm_prefork:notice] [pid 967] AH00163: Apache/2.4.25 (Ubuntu) OpenSSL/1.0.2j configured -- resuming normal operations |
That's what I have in the access-log |
I now switched of http2 and without http2 it works as it should... |
Ah yeah well that should also be a problem with any previous version of the app. Any idea @LukasReschke |
Sorry, I first recognized it after the update to 1.2. |
Maybe we could simply terminate the From what I understand from Line 90 in 5f17e9d
|
Every 5 minutes, the signalling event source is now closed, forcing the client to reconnect (if still available). This should fix #225. Signed-off-by: Joachim Bauch <[email protected]>
Doesn't fix the issue. :-( |
Yeah the problem is the open connection blocks all other requests... so we would need to make sure that one request uses a different connection or something like that, when that is possible... |
Hi, it happened the same to me, is there any planned update with http2 support ? |
just adding that with a fresh install of NC 11, I experienced the same issue (no contacts loading in the left sidebar). works after switching off HTTP2. |
This issue is especially unnerving because enabling http2 shows enormous nextcloud performance gains. |
I do not use Spreedme, but Video Chat from NC 12. |
Any news on a potential fix? Or, perhaps, on what can be done to help with debugging? We are running 2 instances of NC12 (production and testing), both on |
In this case it was not about missing credentials, but we had to change the used technologies. |
This is great news indeed! When can Nextcloud 13 be expected? |
Steps to reproduce
Expected behaviour
Spreed should load as before 1.2
Actual behaviour
see above
Browser
Microphone available: yes/no
yes
Camera available: yes/no
yes
Operating system: Windows/Ubuntu/...
Win 10 Pro
Browser name: Firefox/Chrome/...
firefox and chrome
Browser version: 50.1/55/...
newest
Browser log
jquery-migrate.min.js?v=4d43c3a…:2 JQMIGRATE: Migrate is installed, version 1.4.0
app.js?v=4d43c3a…:356 Starting spreed …
jquery.min.js?v=4d43c3a…:4 Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
send @ jquery.min.js?v=4d43c3a…:4
simplewebrtc.js?v=4d43c3a…:13898 CONNECTED
(anonymous) @ simplewebrtc.js?v=4d43c3a…:13898
webrtc.js?v=4d43c3a…:457 local speaking
Spreed app
Spreed app version: (see apps admin page:
/index.php/settings/apps
)Custom TURN server configured: yes/no (see additional admin settings:
/index.php/settings/admin/additional
)NO
Custom STUN server configured: yes/no (see additional admin settings:
/index.php/settings/admin/additional
)stun.nextcloud.com:443
Server configuration
Operating system: Linux OdOc 3.10.96 #1 SMP PREEMPT Mon May 23 01:50:06 BRT 2016 armv7l
Web server: apache2handler
Database: mysql
PHP version: 7.0.13-0ubuntu0.16.04.1
Nextcloud version: 11.0.1.2
Updated from an older Nextcloud/ownCloud or fresh install:
Updated
Where did you install Nextcloud from:
Signing status:
Signing status
List of activated apps:
App list
The content of config/config.php:
Config report
Are you using external storage, if yes which one: local/smb/sftp/...
No
Are you using encryption: yes/no
No
Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...
The text was updated successfully, but these errors were encountered: