Ensure 3box validation uses the correct address (#7235)

app/scripts/controllers/threebox.js

@@ -20,7 +20,6 @@ class ThreeBoxController {
       addressBookController,
       version,
       getKeyringControllerState,
-      getSelectedAddress,
     } = opts
 
     this.preferencesController = preferencesController
@@ -32,18 +31,20 @@ class ThreeBoxController {
         if (origin !== '3Box') { return [] }
         const isUnlocked = getKeyringControllerState().isUnlocked
 
-        const selectedAddress = getSelectedAddress()
+        const accounts = await this.keyringController.getAccounts()
 
-        if (isUnlocked && selectedAddress) {
+        if (isUnlocked && accounts[0]) {
-          return [selectedAddress]
+          const appKeyAddress = await this.keyringController.getAppKeyAddress(accounts[0], 'wallet://3box.metamask.io')
+          return [appKeyAddress]
         } else {
           return []
         }
       },
-      processPersonalMessage: (msgParams) => {
+      processPersonalMessage: async (msgParams) => {
-        return Promise.resolve(keyringController.signPersonalMessage(msgParams, {
+        const accounts = await this.keyringController.getAccounts()
+        return keyringController.signPersonalMessage({ ...msgParams, from: accounts[0] }, {
           withAppKeyOrigin: 'wallet://3box.metamask.io',
-        }))
+        })
       },
     })
 

app/scripts/metamask-controller.js

@@ -206,7 +206,6 @@ module.exports = class MetamaskController extends EventEmitter {
       keyringController: this.keyringController,
       initState: initState.ThreeBoxController,
       getKeyringControllerState: this.keyringController.memStore.getState.bind(this.keyringController.memStore),
-      getSelectedAddress: this.preferencesController.getSelectedAddress.bind(this.preferencesController),
       version,
     })