- $obj = parent::staticGet(__CLASS__, $k, $v);
- if (!empty($obj)) {
- $obj = Magicsig::fromString($obj->keypair);
-
- // Double check keys: Crypt_RSA did not
- // consistently generate good keypairs.
- // We've also moved to 1024 bit keys.
- if (strlen($obj->publicKey->modulus->toBits()) != 1024) {
- $obj->delete();
- return false;
+ $obj = parent::getKV($k, $v);
+ if ($obj instanceof Magicsig) {
+ $obj->importKeys(); // Loads Crypt_RSA objects etc.
+
+ // Throw out a big fat warning for keys of less than 1024 bits. (
+ // The only case these show up in would be imported or
+ // legacy very-old-StatusNet generated keypairs.
+ if (strlen($obj->publicKey->modulus->toBits()) < 1024) {
+ common_log(LOG_WARNING, sprintf('Salmon key with <1024 bits (%d) belongs to profile with id==%d',
+ strlen($obj->publicKey->modulus->toBits()),
+ $obj->user_id));