From 1d0fdfd957e19a55ef232c21e991cd797c1c3d8f Mon Sep 17 00:00:00 2001 From: fnuesse Date: Wed, 15 May 2019 15:29:06 +0200 Subject: [PATCH] Add null-check to further improve errorchecking for missing compromisedfield for pre 2.3.0 shares Signed-off-by: fnuesse --- js/app/services/shareservice.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js/app/services/shareservice.js b/js/app/services/shareservice.js index 32e54639..195555cf 100644 --- a/js/app/services/shareservice.js +++ b/js/app/services/shareservice.js @@ -191,7 +191,7 @@ try { field_decrypted_value = EncryptService.decryptString(fieldValue, sharedKey); } catch (e) { - if(field === 'compromised'){ + if(field === 'compromised' && fieldValue === null){ //old shares do not have compromised set, so we need to make sure that it will be set for version 2.3.0 //credentials from 2.3.0 onwards have compromised already set, and don't need to worry about that. //if compromised is not the issue, break and throw an error