Fixed #2 and made Signer.sign public.

Really shouldn’t have been otherwise in the first place, honestly.

Author: ahknight

httpsig/sign.py

@@ -60,7 +60,7 @@ def _sign_hmac(self, data):
         hmac.update(data)
         return hmac.digest()
 
-    def _sign(self, data):
+    def sign(self, data):
         if isinstance(data, six.string_types):
             data = data.encode("ascii")
         signed = None
@@ -111,7 +111,7 @@ def sign(self, headers, host=None, method=None, path=None):
         signable = generate_message(
                     required_headers, headers, host, method, path)
 
-        signature = self._sign(signable)
+        signature = super(HeaderSigner, self).sign(signable)
         headers['authorization'] = self.signature_template % signature
 
         return headers

httpsig/tests/test_verify.py

@@ -52,7 +52,7 @@ def test_basic_sign(self):
         BAD = b"this is not the signature you were looking for..."
 
         # generate signed string
-        signature = signer._sign(GOOD)
+        signature = signer.sign(GOOD)
         self.assertTrue(verifier._verify(data=GOOD, signature=signature))
         self.assertFalse(verifier._verify(data=BAD, signature=signature))
 

setup.py

@@ -9,7 +9,6 @@
 
 setup(
     name='httpsig',
-    # version=versioneer.get_version(),
     description="Secure HTTP request signing using the HTTP Signature draft specification",
     long_description=long_description,
     classifiers=[