Skip to content
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

luci-ssl is broken with the latest chrome (likely due to polarssl) #736

Closed
neheb opened this issue Jun 2, 2016 · 12 comments
Closed

luci-ssl is broken with the latest chrome (likely due to polarssl) #736

neheb opened this issue Jun 2, 2016 · 12 comments
Assignees

Comments

@neheb
Copy link
Contributor

neheb commented Jun 2, 2016

works fine with firefox. it just times out at establishing secure connection. no idea why.

i remember there was a news story on how google chrome disabled npn which broke some sites. no idea if luci-ssl also uses npn.

@Vulpecula-nl
Copy link

I have the same problem. Its not working anymore in google chrome version 51.0.2704.84 m (64-bit). I have changed the parameter redirect_https to 0 in /etc/config/uhttpd and now it works again, but not with ssl connection

@hnyman
Copy link
Contributor

hnyman commented Jun 9, 2016

I noticed the same when trying to connect with my Android tablet. No more Luci :-(

EDIT:
it is likely these changes at the end of May:
https://developers.google.com/web/updates/2016/04/chrome-51-deprecations?hl=en

@jow-
Copy link
Contributor

jow- commented Jun 9, 2016

Cannot do much about that. Sounds like something that needs to be addressed in the various crypto backends (ustream-ssl, polar/wolfssl, cyassl/mbedtls, openssl).

@jow-
Copy link
Contributor

jow- commented Jun 9, 2016

Confirmed, swapping libustream-polarssl with libustream-cyassl works, so neither a LuCI, nor a uhttpd issue. Likely needs a library update to polarssl/mbedtls.

@hnyman
Copy link
Contributor

hnyman commented Jun 9, 2016

I tested switching from polarssl to its new incarnation mbedtls, and that works with Chrome, too.
So the culprit it is just the old/ancient polarssl codebase.

Would it be time to switch the default from polarssl to mbedtls?

@hnyman hnyman changed the title luci-ssl is broken with the latest chrome luci-ssl is broken with the latest chrome (likely due to polarssl) Jun 9, 2016
@blogic
Copy link
Contributor

blogic commented Jun 9, 2016

whats the size difference ?

@hnyman
Copy link
Contributor

hnyman commented Jun 9, 2016

Roughly: polarssl 141 kB, cyassl 147 kB, mdebtls 162 kB, openssl 735 kB

from ar71xx:

libcyassl_3.9.0-1_mips_34kc.ipk                    08-Jun-2016 07:46              147593
libmbedtls_2.2.1-1_mips_34kc.ipk                   08-Jun-2016 07:33              162085
libopenssl_1.0.2h-1_mips_34kc.ipk                  08-Jun-2016 07:49              735634
libpolarssl_1.3.16-1_mips_34kc.ipk                 08-Jun-2016 07:33              141441

libustream-cyassl_2016-06-07-17085b7abc0cd09c64..> 08-Jun-2016 08:07                3842
libustream-mbedtls_2016-06-07-17085b7abc0cd09c6..> 08-Jun-2016 08:07                4336
libustream-openssl_2016-06-07-17085b7abc0cd09c6..> 08-Jun-2016 08:07                4146
libustream-polarssl_2016-06-07-17085b7abc0cd09c..> 08-Jun-2016 08:07                4233

@hnyman
Copy link
Contributor

hnyman commented Jun 9, 2016

Note that switching from polarssl to cyassl or mbedtls would have impact on a few non-core packages, as they do not have variants / config options for non-polarssl. (However, most apps using polarssl like transmission, umurmur, openvpn and shairport have also openssl versions, so they have a backup option.)

px5g, the creator of the self-signed certificate for Luci / uhttpd, requires polarssl at the moment. That is probably the item on the critical path to change away from polarssl. As long as px5g only supports polarssl, there is no practical way to switch away from it.

Ps. polarssl was ntoed to be difficult already a few motnsh ago:
https://dev.openwrt.org/ticket/22173#comment:1

@jow-
Copy link
Contributor

jow- commented Jun 11, 2016

I believe the issuse is solved with lede-project/source@9e45f9d

@neheb
Copy link
Contributor Author

neheb commented Jun 15, 2016

Issue is fixed. That's really bizarre that GCM is required now. Maybe google is more aggressive with TLS 1.2? A different solution would probably be to disable TLS 1.2 and keep 1.1. GCM is not available with 1.1. I'm not sure if size is a real concern here.

@neheb neheb closed this as completed Jun 15, 2016
@rvalles
Copy link

rvalles commented Jun 15, 2016

Fixed in lede, but not openwrt?

dircleaned, pulled, built and installed designated driver right now, issue still present.

@Dekker500
Copy link

OpenWrt Designated Driver snapshot r49388 from June 21 still exhibits the symptoms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants