D8354: Remove check for SNI hosts when chosing

2017-10-19 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R289:588151c5e000: Remove check for SNI hosts when chosing whether to use legacy mode (authored by davidedmundson). REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE

D8354: Remove check for SNI hosts when chosing

2017-10-18 Thread David Edmundson
davidedmundson added a comment. > Just tested: this fixes the freeze I'm experiencing on startup! It's a combo of both this, the other one, and apparently your change from a while ago with the async version lookup. I'm quite glad there isn't a forth issue! I suggest us devs export

D8354: Remove check for SNI hosts when chosing

2017-10-18 Thread Martin Flöser
graesslin accepted this revision. graesslin added a comment. This revision is now accepted and ready to land. Just tested: this fixes the freeze I'm experiencing on startup! REPOSITORY R289 KNotifications BRANCH master REVISION DETAIL https://phabricator.kde.org/D8354 To:

D8354: Remove check for SNI hosts when chosing

2017-10-18 Thread David Edmundson
davidedmundson created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Previous code (since very first commit) checked for both having a valid KSNI watcher as well as making sure there was a host