From 220c61f1242659fbdd0efdb121f4b1167f9ea61c Mon Sep 17 00:00:00 2001
From: Seth Schoen <schoen@eff.org>
Date: Mon, 2 Feb 2015 17:03:29 -0800
Subject: [PATCH] Small fixes to how errors are reported

---
 letsencrypt/client/standalone_authenticator.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/letsencrypt/client/standalone_authenticator.py b/letsencrypt/client/standalone_authenticator.py
index 17cddb5a3..1ebd91f3a 100644
--- a/letsencrypt/client/standalone_authenticator.py
+++ b/letsencrypt/client/standalone_authenticator.py
@@ -135,7 +135,7 @@ def tls_parse_client_hello(tls_record):
             if first_sn_type != "\0":
                 # SNI extension referenced something other than a
                 # hostname
-                return False
+                return None
             first_sn_length = unpack_2bytes(handshake[i+3:i+5])
             first_sn = handshake[i+5:i+5+first_sn_length]
             return best_ciphersuite, first_sn
@@ -491,7 +491,7 @@ class StandaloneAuthenticator(object):
                 del self.tasks[nonce + CONFIG.INVALID_EXT]
             else:
                 # Could not find the challenge to remove!
-                assert False
+                raise ValueError("could not find the challenge to remove")
         if self.child_pid and not self.tasks:
             # There are no remaining challenges, so
             # try to shutdown self.child_pid cleanly.