mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Additional logging when we can't decrypt a user's profile (#1483)

Browse source 
Found a number of 'Illegal buffer' errors in an Electron log submitted
today. As far as I can tell, these same profiles are succeedig for me,
so it's time to collect more data.

FREEBIE
This commit is contained in:
Scott Nonnenberg 2017-09-18 15:33:08 -07:00 • committed by GitHub
parent c420c597f9
commit 2786baf2b5
1 changed files with 20 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

30
js/models/conversations.js
View file

@ -816,7 +816,12 @@
// fail
if (e.name === 'ProfileDecryptError') {
// probably the profile key has changed.
console.log('decryptProfile error:', e);
console.log(
'decryptProfile error:',
id,
profile,
error && error.stack ? error.stack : error
);
}
});
}.bind(this));
@ -831,18 +836,23 @@
var key = this.get('profileKey');
if (!key) { return; }
// decode
var data = dcodeIO.ByteBuffer.wrap(encryptedName, 'base64').toArrayBuffer();
try {
// decode
var data = dcodeIO.ByteBuffer.wrap(encryptedName, 'base64').toArrayBuffer();
// decrypt
return textsecure.crypto.decryptProfileName(data, key).then(function(decrypted) {
// decrypt
return textsecure.crypto.decryptProfileName(data, key).then(function(decrypted) {
// encode
var name = dcodeIO.ByteBuffer.wrap(decrypted).toString('utf8');
// encode
var name = dcodeIO.ByteBuffer.wrap(decrypted).toString('utf8');
// set
this.set({profileName: name});
}.bind(this));
// set
this.set({profileName: name});
}.bind(this));
}
catch (e) {
return Promise.reject(e);
}
},
setProfileAvatar: function(avatarPath) {
if (!avatarPath) { return; }