OS X and Chrome change js doesn't change field visibility correctly
Choosing AES then 3DES on IKEv2 Phase 1 leaves key length selector active, resulting in a broken config.
I was able to duplicate this using Mac OS X 10.11.5 with Google Chrome Version 50.0.2661.102 (64-bit) after seeing a "3DES (256-bit)" setting on a customer system. It seems to be only Google Chrome and only IKEv2.
Workaround: Re-edit without changing from 3DES and saving corrects the config.
(cherry picked from commit fe68a6a2b28c897cb3a8f8fda452c25f649556f1)
#2 Updated by Chris Buechler about 4 years ago
- Subject changed from Choosing AES then 3DES on IKEv2 Phase 1 leaves key length selector active, resulting in a broken config. to OS X and Chrome change js doesn't change field visibility correctly
- Status changed from New to Confirmed
- Affected Version changed from 2.3.1 to 2.3.x
vpn_ipsec_phase1.php - If you edit it with AES 256 chosen, then select 3DES, it leaves the 256 box there initially. Then click back on the ealgo box, it blinks, and the ealgo_keylen box is hidden. Click 3DES again to close the ealgo box and it remains correctly hidden. Similar then if you change back to AES, initially there is no keylen box. Click ealgo and it blinks and the keylen box appears.
Similar if you switch from IKEv1 to v2 or vice-versa and look for the hide/show of 'Negotiation mode' field.
vpn_openvpn_server.php - for instance change the server mode between shared key and SSL/TLS modes and look for the appropriate Cryptographic Settings fields changing (or not). Same behavior as described for vpn_ipsec_phase1.php, if you click back in the box it blinks and sets the fields appropriately.
#3 Updated by Steve Beaver about 4 years ago
- Status changed from Confirmed to Feedback
- Assignee changed from Steve Beaver to Chris Linstruth
This issue is caused by the way Chrome handles "click" events bound to select inputs.
Changing the JQuery from "click" to "change" fixes the issue and works across all browsers. e.g.:
- $('#mode').click() => $('#mode').change()
The instances reported here have been updated. I'll check all other pages as time permits.