diff --git a/src/app/tools/exposed-passwords-report.component.ts b/src/app/tools/exposed-passwords-report.component.ts index 8d210007b9b..b1d6cde0d29 100644 --- a/src/app/tools/exposed-passwords-report.component.ts +++ b/src/app/tools/exposed-passwords-report.component.ts @@ -43,7 +43,7 @@ export class ExposedPasswordsReportComponent extends CipherReportComponent imple const exposedPasswordCiphers: CipherView[] = []; const promises: Promise[] = []; allCiphers.forEach((c) => { - if (c.type !== CipherType.Login || c.login.password == null || c.login.password === '') { + if (c.type !== CipherType.Login || c.login.password == null || c.login.password === '' || c.isDeleted) { return; } const promise = this.auditService.passwordLeaked(c.login.password).then((exposedCount) => { diff --git a/src/app/tools/inactive-two-factor-report.component.ts b/src/app/tools/inactive-two-factor-report.component.ts index 29cc91919f2..11042ed5bd5 100644 --- a/src/app/tools/inactive-two-factor-report.component.ts +++ b/src/app/tools/inactive-two-factor-report.component.ts @@ -45,7 +45,8 @@ export class InactiveTwoFactorReportComponent extends CipherReportComponent impl const promises: Promise[] = []; const docs = new Map(); allCiphers.forEach((c) => { - if (c.type !== CipherType.Login || (c.login.totp != null && c.login.totp !== '') || !c.login.hasUris) { + if (c.type !== CipherType.Login || (c.login.totp != null && c.login.totp !== '') || !c.login.hasUris || + c.isDeleted) { return; } for (let i = 0; i < c.login.uris.length; i++) { diff --git a/src/app/tools/reused-passwords-report.component.ts b/src/app/tools/reused-passwords-report.component.ts index 06d0cccf9c6..cb935acf6ef 100644 --- a/src/app/tools/reused-passwords-report.component.ts +++ b/src/app/tools/reused-passwords-report.component.ts @@ -37,7 +37,7 @@ export class ReusedPasswordsReportComponent extends CipherReportComponent implem const ciphersWithPasswords: CipherView[] = []; this.passwordUseMap = new Map(); allCiphers.forEach((c) => { - if (c.type !== CipherType.Login || c.login.password == null || c.login.password === '') { + if (c.type !== CipherType.Login || c.login.password == null || c.login.password === '' || c.isDeleted) { return; } ciphersWithPasswords.push(c); diff --git a/src/app/tools/unsecured-websites-report.component.ts b/src/app/tools/unsecured-websites-report.component.ts index 482e0c5d330..3648d23a7a4 100644 --- a/src/app/tools/unsecured-websites-report.component.ts +++ b/src/app/tools/unsecured-websites-report.component.ts @@ -33,7 +33,7 @@ export class UnsecuredWebsitesReportComponent extends CipherReportComponent impl async setCiphers() { const allCiphers = await this.getAllCiphers(); const unsecuredCiphers = allCiphers.filter((c) => { - if (c.type !== CipherType.Login || !c.login.hasUris) { + if (c.type !== CipherType.Login || !c.login.hasUris || c.isDeleted) { return false; } return c.login.uris.some((u) => u.uri != null && u.uri.indexOf('http://') === 0); diff --git a/src/app/tools/weak-passwords-report.component.ts b/src/app/tools/weak-passwords-report.component.ts index e8bf3639cd6..17982e64f79 100644 --- a/src/app/tools/weak-passwords-report.component.ts +++ b/src/app/tools/weak-passwords-report.component.ts @@ -40,7 +40,7 @@ export class WeakPasswordsReportComponent extends CipherReportComponent implemen const allCiphers = await this.getAllCiphers(); const weakPasswordCiphers: CipherView[] = []; allCiphers.forEach((c) => { - if (c.type !== CipherType.Login || c.login.password == null || c.login.password === '') { + if (c.type !== CipherType.Login || c.login.password == null || c.login.password === '' || c.isDeleted) { return; } const hasUsername = c.login.username != null && c.login.username.trim() !== '';