Add types for addLogin and changePassword runtime messages

src/background/models/addLoginRuntimeMessage.ts

@@ -0,0 +1,5 @@
+export default class AddLoginRuntimeMessage {
+    username: string;
+    password: string;
+    url: string;
+}

src/background/models/changePasswordRuntimeMessage.ts

@@ -0,0 +1,5 @@
+export default class ChangePasswordRuntimeMessage {
+    currentPassword: string;
+    newPassword: string;
+    url: string;
+}

src/background/notification.background.ts

@@ -22,6 +22,8 @@ import { PolicyType } from 'jslib-common/enums/policyType';
 
 import AddChangePasswordQueueMessage from './models/addChangePasswordQueueMessage';
 import AddLoginQueueMessage from './models/addLoginQueueMessage';
+import AddLoginRuntimeMessage from './models/addLoginRuntimeMessage';
+import ChangePasswordRuntimeMessage from './models/changePasswordRuntimeMessage';
 import LockedVaultPendingNotificationsItem from './models/lockedVaultPendingNotificationsItem';
 import { NotificationQueueMessageType } from './models/NotificationQueueMessageType';
 
@@ -178,7 +180,7 @@ export default class NotificationBackground {
         }
     }
 
-    private async addLogin(loginInfo: any, tab: chrome.tabs.Tab) {
+    private async addLogin(loginInfo: AddLoginRuntimeMessage, tab: chrome.tabs.Tab) {
         const loginDomain = Utils.getDomain(loginInfo.url);
         if (loginDomain == null) {
             return;
@@ -220,7 +222,7 @@ export default class NotificationBackground {
         }
     }
 
-    private async pushAddLoginToQueue(loginDomain: string, loginInfo: any, tab: chrome.tabs.Tab, isVaultLocked: boolean = false) {
+    private async pushAddLoginToQueue(loginDomain: string, loginInfo: AddLoginRuntimeMessage, tab: chrome.tabs.Tab, isVaultLocked: boolean = false) {
         // remove any old messages for this tab
         this.removeTabFromNotificationQueue(tab);
         const message: AddLoginQueueMessage = {
@@ -237,7 +239,7 @@ export default class NotificationBackground {
         await this.checkNotificationQueue(tab);
     }
 
-    private async changedPassword(changeData: any, tab: chrome.tabs.Tab) {
+    private async changedPassword(changeData: ChangePasswordRuntimeMessage, tab: chrome.tabs.Tab) {
         const loginDomain = Utils.getDomain(changeData.url);
         if (loginDomain == null) {
             return;

src/content/notificationBar.ts

@@ -1,3 +1,6 @@
+import AddLoginRuntimeMessage from 'src/background/models/addLoginRuntimeMessage';
+import ChangePasswordRuntimeMessage from 'src/background/models/changePasswordRuntimeMessage';
+
 document.addEventListener('DOMContentLoaded', event => {
     if (window.location.hostname.indexOf('vault.bitwarden.com') > -1) {
         return;
@@ -294,7 +297,7 @@ document.addEventListener('DOMContentLoaded', event => {
             }
             const disabledBoth = disabledChangedPasswordNotification && disabledAddLoginNotification;
             if (!disabledBoth && formData[i].usernameEl != null && formData[i].passwordEl != null) {
-                const login = {
+                const login: AddLoginRuntimeMessage = {
                     username: formData[i].usernameEl.value,
                     password: formData[i].passwordEl.value,
                     url: document.URL,
@@ -343,13 +346,15 @@ document.addEventListener('DOMContentLoaded', event => {
 
                 if (newPass != null && curPass != null || (newPassOnly && newPass != null)) {
                     processedForm(form);
+
+                    const changePasswordRuntimeMessage: ChangePasswordRuntimeMessage = {
+                        newPassword: newPass,
+                        currentPassword: curPass,
+                        url: document.URL,
+                    };
                     sendPlatformMessage({
                         command: 'bgChangedPassword',
-                        data: {
+                        data: changePasswordRuntimeMessage,
-                            newPassword: newPass,
-                            currentPassword: curPass,
-                            url: document.URL,
-                        },
                     });
                     break;
                 }