jonny
/
Signal-Desktop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sendToGroup/_shouldFailSend: Don't fail send on 401 

Co-authored-by: Scott Nonnenberg <scott@signal.org>
This commit is contained in:
automated-signal 2022-10-12 16:14:18 -07:00 committed by GitHub
parent 3e3e5d6210
commit 3708d89ade
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ts/test-electron/util/sendToGroup_test.ts
View File

@ -233,7 +233,7 @@ describe('sendToGroup', () => {
it('returns true for a specified error codes', () => {
// eslint-disable-next-line @typescript-eslint/no-explicit-any
const error: any = new Error('generic');
error.code = 401;
error.code = 428;
assert.isTrue(_shouldFailSend(error, 'testing generic'));
assert.isTrue(
@ -321,7 +321,7 @@ describe('sendToGroup', () => {
// eslint-disable-next-line @typescript-eslint/no-explicit-any
const error: any = new Error('generic');
error.code = 401;
error.code = 428;
assert.isTrue(
_shouldFailSend(

5
ts/util/sendToGroup.ts
View File

@ -787,11 +787,6 @@ export function _shouldFailSend(error: unknown, logId: string): boolean {
return true;
}
if (error.code === 401) {
logError('Permissions error, failing.');
return true;
}
if (error.code === 404) {
logError('Missing user or endpoint error, failing.');
return true;