diff options
author | Brion Vibber <brion@pobox.com> | 2010-03-04 07:00:45 -0800 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-03-04 07:00:45 -0800 |
commit | 62d5f1addb1b710a387f7732ace1dd33bd1b86c1 (patch) | |
tree | df1d6d402dd057c330b0eb71b803d595c024d476 /plugins/OStatus/lib/magicenvelope.php | |
parent | 0f1c6e239acbf2c024c3aae9819d891f02b05bfd (diff) | |
parent | 9f861e9d895325adbb2dc7f1d540a442be2c1b2f (diff) |
Merge branch 'testing'
Diffstat (limited to 'plugins/OStatus/lib/magicenvelope.php')
-rw-r--r-- | plugins/OStatus/lib/magicenvelope.php | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/plugins/OStatus/lib/magicenvelope.php b/plugins/OStatus/lib/magicenvelope.php index 230d81ba1..fb8c57c71 100644 --- a/plugins/OStatus/lib/magicenvelope.php +++ b/plugins/OStatus/lib/magicenvelope.php @@ -156,18 +156,32 @@ class MagicEnvelope public function verify($env) { if ($env['alg'] != 'RSA-SHA256') { + common_log(LOG_DEBUG, "Salmon error: bad algorithm"); return false; } if ($env['encoding'] != MagicEnvelope::ENCODING) { + common_log(LOG_DEBUG, "Salmon error: bad encoding"); return false; } $text = base64_decode($env['data']); $signer_uri = $this->getAuthor($text); - $verifier = Magicsig::fromString($this->getKeyPair($signer_uri)); + try { + $keypair = $this->getKeyPair($signer_uri); + } catch (Exception $e) { + common_log(LOG_DEBUG, "Salmon error: ".$e->getMessage()); + return false; + } + + $verifier = Magicsig::fromString($keypair); + if (!$verifier) { + common_log(LOG_DEBUG, "Salmon error: unable to parse keypair"); + return false; + } + return $verifier->verify($env['data'], $env['sig']); } |