Desktop: Desktop App issues certificate errors since v3.7.1

Created on 26 Mar 2018  路  7Comments  路  Source: mattermost/desktop

I confirm (by marking "x" in the [ ] below: [x]):


Summary
Desktop App issues certificate errors since v3.7.1

Steps to reproduce

There are no clear reproduction steps, but at least three separate users have reported certificate errors since v3.7.1:

https://pre-release.mattermost.com/core/pl/4jmwur5fr3no9j1nhg6ntgyfzr
https://pre-release.mattermost.com/core/pl/fyfek1upt7g83jxbh4hfwimf9y
https://pre-release.mattermost.com/private-core/pl/1tcwauk7ibdr9rf1gcz6pf8tur

All Platforms Electron Priorit1 TypBug

All 7 comments

@yuya-oc I know you're already looking into this, but created a ticket for us to track.

@amyblais Added a priority 1 label since it affects the "Trusted" principle.

Probably related: https://github.com/electron/electron/issues/11997 And it might be fixed at Electron 1.8.3.

As an ad-hoc workaround, we might be able to ignore the error by using setVertificateVerifyProc().

@amyblais Can you help ask Lindy to download a test build from https://github.com/mattermost/desktop/pull/741 which upgrades Electron to 1.8.4.

If it resolves it for her, then at least we'll know it's related.

@jasonblais, @yuya-oc FYI, Lindy reported that she hasn't seen the issues anymore with the test build from https://github.com/mattermost/desktop/pull/741.

Great! That means it should already be fixed for 4.1.0.

@amyblais Can you also help Lindy to download the test artifact here, which should fix it for 4.0.1: https://github.com/mattermost/desktop/pull/743

Lindy tested with #743 and saw no issues.
Linda also reported that she has seen no issues with the test build from #741 on Mac.

Perfect. @yuya-oc We can merge #743 and the changelog PR once you've reviewed (#744).

We can then close https://github.com/mattermost/desktop/issues/742 and cut 4.0.1-final.

Was this page helpful?
0 / 5 - 0 ratings