aaron/mattermost-desktop
1
0
Fork 0
Code Issues Pull requests Actions 1 Packages Projects Releases Wiki Activity

[MM-56882] Fix a potential crash in diagnostics (#2960)

Browse source
This commit is contained in:
Devin Binnie 2024-02-16 12:27:30 -05:00 committed by GitHub
parent ad3da23863
commit 405403ca99
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/main/diagnostics/steps/internal/utils.ts
View file

@ -72,11 +72,15 @@ export async function isOnline(logger: ElectronLog = log, url = IS_ONLINE_ENDPOI
resp.on('end', () => { resp.on('end', () => {
logger.debug('resp.on.end', {data, url}); logger.debug('resp.on.end', {data, url});
if (data.length) { if (data.length) {
try {
const respBody = JSON.parse(data); const respBody = JSON.parse(data);
if (respBody.status === 'OK') { if (respBody.status === 'OK') {
resolve(true); resolve(true);
return; return;
} }
} catch (e) {
logger.error('Cannot parse response')
}
} }
resolve(false); resolve(false);
}); });