use .ok field on serverRequest to check for request success

pull/1383/head
Audric Ackermann 5 years ago
parent 2b13321c23
commit 5de2dea407
No known key found for this signature in database
GPG Key ID: 999F434D76324AD4

@ -101,7 +101,7 @@
const res = await window.tokenlessFileServerAdnAPI.serverRequest( const res = await window.tokenlessFileServerAdnAPI.serverRequest(
'loki/v1/time' 'loki/v1/time'
); );
if (res.statusCode === 200) { if (res.ok) {
timestamp = res.response; timestamp = res.response;
} }
} catch (e) { } catch (e) {

@ -243,6 +243,7 @@ const serverRequest = async (endpoint, options = {}) => {
); );
return { return {
err: e, err: e,
ok: false,
}; };
} }
@ -313,6 +314,7 @@ const serverRequest = async (endpoint, options = {}) => {
return { return {
err: e, err: e,
ok: false,
}; };
} }
@ -320,6 +322,7 @@ const serverRequest = async (endpoint, options = {}) => {
return { return {
err: 'noResult', err: 'noResult',
response, response,
ok: false,
}; };
} }
@ -336,11 +339,13 @@ const serverRequest = async (endpoint, options = {}) => {
err: 'statusCode', err: 'statusCode',
statusCode: result.status, statusCode: result.status,
response, response,
ok: false,
}; };
} }
return { return {
statusCode: result.status, statusCode: result.status,
response, response,
ok: result.status >= 200 && result.status <= 299,
}; };
}; };
@ -692,7 +697,7 @@ class LokiAppDotNetServerAPI {
} }
return null; return null;
} }
if (res.statusCode !== 200) { if (!res.ok) {
log.error('requestToken request failed'); log.error('requestToken request failed');
return null; return null;
} }
@ -712,7 +717,7 @@ class LokiAppDotNetServerAPI {
}, },
noJson: true, noJson: true,
}); });
return res.statusCode === 200; return res.ok;
} catch (e) { } catch (e) {
log.error('submitToken serverRequest failure', e.code, e.message); log.error('submitToken serverRequest failure', e.code, e.message);
return false; return false;
@ -937,12 +942,9 @@ class LokiAppDotNetServerAPI {
rawBody: data, rawBody: data,
}; };
const { statusCode, response } = await this.serverRequest( const { response, ok } = await this.serverRequest(endpoint, options);
endpoint,
options
);
if (statusCode !== 200) { if (!ok) {
throw new Error(`Failed to upload avatar to ${this.baseServerUrl}`); throw new Error(`Failed to upload avatar to ${this.baseServerUrl}`);
} }
@ -970,11 +972,8 @@ class LokiAppDotNetServerAPI {
rawBody: data, rawBody: data,
}; };
const { statusCode, response } = await this.serverRequest( const { ok, response } = await this.serverRequest(endpoint, options);
endpoint, if (!ok) {
options
);
if (statusCode !== 200) {
throw new Error(`Failed to upload data to server: ${this.baseServerUrl}`); throw new Error(`Failed to upload data to server: ${this.baseServerUrl}`);
} }

@ -438,8 +438,7 @@ const sendOnionRequest = async (
id id
); );
onionPayload += payload.length; onionPayload += payload.length;
log.debug('Onion payload size: ', payload.length); log.debug('Onion payload size: ', payload.length, ' total:', onionPayload);
log.debug('Onion total: ', onionPayload);
const guardFetchOptions = { const guardFetchOptions = {
method: 'POST', method: 'POST',

Loading…
Cancel
Save