Throw error if more than one device has same csr in container

Summary:
Getting more visibility on this.

We're seeing cases where the wrong deviceId is getting stored in the device connection_config.json.

This comes from flipper desktop in the response of 'signCertificate', and is figured out using getTargetDeviceId.

If we ever see these logs then something is wrong, and seeing the csr should help.

Reviewed By: passy

Differential Revision: D13357882

fbshipit-source-id: eb6befe07e7f9e7606e8940a1fed619351946d37
This commit is contained in:
John Knox
2018-12-06 05:35:23 -08:00
committed by Facebook Github Bot
parent 3b18303560
commit ee23119f24

View File

@@ -281,6 +281,12 @@ export default class CertificateProvider {
`No matching device found for app: ${appName}`,
);
}
if (matchingIds.length > 1) {
console.error(
new RecurringError('More than one matching device found for CSR'),
csr,
);
}
return matchingIds[0];
});
});