Do not raise on not 2XX error codes
authorTomás Touceda <chiiph@leap.se>
Thu, 29 Aug 2013 20:41:43 +0000 (17:41 -0300)
committerTomás Touceda <chiiph@leap.se>
Fri, 30 Aug 2013 18:05:38 +0000 (15:05 -0300)
Also accept unicode ascii keys

changes/bug_minor_issues [new file with mode: 0644]
src/leap/keymanager/__init__.py
src/leap/keymanager/openpgp.py

diff --git a/changes/bug_minor_issues b/changes/bug_minor_issues
new file mode 100644 (file)
index 0000000..91a69ed
--- /dev/null
@@ -0,0 +1,4 @@
+  o Do not raise exception when a GET request doesn't return 2XX
+    code. Nickserver uses error codes for more verbosity in the
+    result.
+  o Accept unicode ascii keys along with str.
\ No newline at end of file
index 05cbcca..2f39ad9 100644 (file)
@@ -127,11 +127,16 @@ class KeyManager(object):
             self._ca_cert_path is not None,
             'We need the CA certificate path!')
         res = self._fetcher.get(uri, data=data, verify=self._ca_cert_path)
-        # assert that the response is valid
-        res.raise_for_status()
-        leap_assert(
-            res.headers['content-type'].startswith('application/json'),
-            'Content-type is not JSON.')
+        # Nickserver now returns 404 for key not found and 500 for
+        # other cases (like key too small), so we are skipping this
+        # check for the time being
+        # res.raise_for_status()
+
+        # Responses are now text/plain, although it's json anyway, but
+        # this will fail when it shouldn't
+        # leap_assert(
+        #     res.headers['content-type'].startswith('application/json'),
+        #     'Content-type is not JSON.')
         return res
 
     def _put(self, uri, data=None):
index aa04ed0..7946db8 100644 (file)
@@ -301,9 +301,9 @@ class OpenPGPScheme(EncryptionScheme):
         Put key contained in ascii-armored C{key_data} in local storage.
 
         :param key_data: The key data to be stored.
-        :type key_data: str
+        :type key_data: str or unicode
         """
-        leap_assert_type(key_data, str)
+        leap_assert_type(key_data, (str, unicode))
         # TODO: add more checks for correct key data.
         leap_assert(key_data is not None, 'Data does not represent a key.')