mirror of
https://github.com/pyauth/requests-http-signature.git
synced 2024-12-21 12:37:11 +01:00
Use isort and black
This commit is contained in:
parent
c643c8ccd7
commit
45f540a22c
6 changed files with 116 additions and 86 deletions
2
.github/workflows/ci.yml
vendored
2
.github/workflows/ci.yml
vendored
|
@ -24,3 +24,5 @@ jobs:
|
|||
run: pip install .[tests]
|
||||
- name: Test
|
||||
run: make test
|
||||
- uses: isort/isort-action@v1.0.0
|
||||
- uses: psf/black@stable
|
||||
|
|
|
@ -1,7 +1,9 @@
|
|||
import os, sys
|
||||
import os
|
||||
import sys
|
||||
|
||||
import guzzle_sphinx_theme
|
||||
|
||||
sys.path.insert(0, os.path.abspath('..'))
|
||||
sys.path.insert(0, os.path.abspath(".."))
|
||||
|
||||
project = "requests-http-signature"
|
||||
copyright = "Andrey Kislyuk"
|
||||
|
@ -27,6 +29,6 @@ html_sidebars = {
|
|||
"logo-text.html",
|
||||
# "globaltoc.html",
|
||||
"localtoc.html",
|
||||
"searchbox.html"
|
||||
"searchbox.html",
|
||||
]
|
||||
}
|
||||
|
|
5
pyproject.toml
Normal file
5
pyproject.toml
Normal file
|
@ -0,0 +1,5 @@
|
|||
[tool.black]
|
||||
line-length = 120
|
||||
exclude = ".*/version.py"
|
||||
[tool.isort]
|
||||
profile = "black"
|
|
@ -89,16 +89,19 @@ class HTTPSignatureAuth(requests.auth.AuthBase):
|
|||
|
||||
_auto_cover_header_fields = {"authorization", "content-digest", "date"}
|
||||
|
||||
def __init__(self, *,
|
||||
signature_algorithm: Type[HTTPSignatureAlgorithm],
|
||||
key: bytes = None,
|
||||
key_id: str,
|
||||
key_resolver: HTTPSignatureKeyResolver = None,
|
||||
covered_component_ids: Sequence[str] = ("@method", "@authority", "@target-uri"),
|
||||
label: str = None,
|
||||
include_alg: bool = True,
|
||||
use_nonce: bool = False,
|
||||
expires_in: datetime.timedelta = None):
|
||||
def __init__(
|
||||
self,
|
||||
*,
|
||||
signature_algorithm: Type[HTTPSignatureAlgorithm],
|
||||
key: bytes = None,
|
||||
key_id: str,
|
||||
key_resolver: HTTPSignatureKeyResolver = None,
|
||||
covered_component_ids: Sequence[str] = ("@method", "@authority", "@target-uri"),
|
||||
label: str = None,
|
||||
include_alg: bool = True,
|
||||
use_nonce: bool = False,
|
||||
expires_in: datetime.timedelta = None,
|
||||
):
|
||||
if key_resolver is None and key is None:
|
||||
raise RequestsHttpSignatureException("Either key_resolver or key must be specified.")
|
||||
if key_resolver is not None and key is not None:
|
||||
|
@ -112,9 +115,11 @@ class HTTPSignatureAuth(requests.auth.AuthBase):
|
|||
self.use_nonce = use_nonce
|
||||
self.covered_component_ids = covered_component_ids
|
||||
self.expires_in = expires_in
|
||||
self.signer = HTTPMessageSigner(signature_algorithm=signature_algorithm,
|
||||
key_resolver=key_resolver,
|
||||
component_resolver_class=self.component_resolver_class)
|
||||
self.signer = HTTPMessageSigner(
|
||||
signature_algorithm=signature_algorithm,
|
||||
key_resolver=key_resolver,
|
||||
component_resolver_class=self.component_resolver_class,
|
||||
)
|
||||
|
||||
def add_date(self, request, timestamp):
|
||||
if "Date" not in request.headers:
|
||||
|
@ -156,14 +161,16 @@ class HTTPSignatureAuth(requests.auth.AuthBase):
|
|||
created = self.get_created(request)
|
||||
expires = self.get_expires(request, created=created)
|
||||
covered_component_ids = self.get_covered_component_ids(request)
|
||||
self.signer.sign(request,
|
||||
key_id=self.key_id,
|
||||
created=created,
|
||||
expires=expires,
|
||||
nonce=self.get_nonce(request),
|
||||
label=self.label,
|
||||
include_alg=self.include_alg,
|
||||
covered_component_ids=covered_component_ids)
|
||||
self.signer.sign(
|
||||
request,
|
||||
key_id=self.key_id,
|
||||
created=created,
|
||||
expires=expires,
|
||||
nonce=self.get_nonce(request),
|
||||
label=self.label,
|
||||
include_alg=self.include_alg,
|
||||
covered_component_ids=covered_component_ids,
|
||||
)
|
||||
return request
|
||||
|
||||
@classmethod
|
||||
|
@ -173,11 +180,15 @@ class HTTPSignatureAuth(requests.auth.AuthBase):
|
|||
return message.body
|
||||
|
||||
@classmethod
|
||||
def verify(cls, message: Union[requests.PreparedRequest, requests.Response], *,
|
||||
require_components: Sequence[str] = ("@method", "@authority", "@target-uri"),
|
||||
signature_algorithm: Type[HTTPSignatureAlgorithm],
|
||||
key_resolver: HTTPSignatureKeyResolver,
|
||||
max_age: datetime.timedelta = datetime.timedelta(days=1)) -> VerifyResult:
|
||||
def verify(
|
||||
cls,
|
||||
message: Union[requests.PreparedRequest, requests.Response],
|
||||
*,
|
||||
require_components: Sequence[str] = ("@method", "@authority", "@target-uri"),
|
||||
signature_algorithm: Type[HTTPSignatureAlgorithm],
|
||||
key_resolver: HTTPSignatureKeyResolver,
|
||||
max_age: datetime.timedelta = datetime.timedelta(days=1),
|
||||
) -> VerifyResult:
|
||||
"""
|
||||
Verify an HTTP message signature.
|
||||
|
||||
|
@ -244,9 +255,11 @@ class HTTPSignatureAuth(requests.auth.AuthBase):
|
|||
if "content-digest" not in require_components and '"content-digest"' not in require_components:
|
||||
require_components = list(require_components) + ["content-digest"]
|
||||
|
||||
verifier = HTTPMessageVerifier(signature_algorithm=signature_algorithm,
|
||||
key_resolver=key_resolver,
|
||||
component_resolver_class=cls.component_resolver_class)
|
||||
verifier = HTTPMessageVerifier(
|
||||
signature_algorithm=signature_algorithm,
|
||||
key_resolver=key_resolver,
|
||||
component_resolver_class=cls.component_resolver_class,
|
||||
)
|
||||
verify_results = verifier.verify(message, max_age=max_age)
|
||||
if len(verify_results) != 1:
|
||||
raise InvalidSignature("Multiple signatures are not supported.")
|
||||
|
|
48
setup.py
48
setup.py
|
@ -1,24 +1,20 @@
|
|||
#!/usr/bin/env python
|
||||
|
||||
from setuptools import setup, find_packages
|
||||
from setuptools import find_packages, setup
|
||||
|
||||
setup(
|
||||
name='requests-http-signature',
|
||||
url='https://github.com/pyauth/requests-http-signature',
|
||||
license='Apache Software License',
|
||||
author='Andrey Kislyuk',
|
||||
author_email='kislyuk@gmail.com',
|
||||
name="requests-http-signature",
|
||||
url="https://github.com/pyauth/requests-http-signature",
|
||||
license="Apache Software License",
|
||||
author="Andrey Kislyuk",
|
||||
author_email="kislyuk@gmail.com",
|
||||
description="A Requests auth module for HTTP Message Signatures",
|
||||
long_description=open('README.rst').read(),
|
||||
long_description=open("README.rst").read(),
|
||||
use_scm_version={
|
||||
"write_to": "requests_http_signature/version.py",
|
||||
},
|
||||
setup_requires=['setuptools_scm >= 3.4.3'],
|
||||
install_requires=[
|
||||
"http-message-signatures >= 0.4.3",
|
||||
"http-sfv >= 0.9.3",
|
||||
"requests >= 2.25.1"
|
||||
],
|
||||
setup_requires=["setuptools_scm >= 3.4.3"],
|
||||
install_requires=["http-message-signatures >= 0.4.3", "http-sfv >= 0.9.3", "requests >= 2.25.1"],
|
||||
extras_require={
|
||||
"tests": [
|
||||
"flake8",
|
||||
|
@ -29,23 +25,23 @@ setup(
|
|||
"types-requests",
|
||||
]
|
||||
},
|
||||
packages=find_packages(exclude=['test']),
|
||||
packages=find_packages(exclude=["test"]),
|
||||
include_package_data=True,
|
||||
package_data={
|
||||
"http_message_signatures": ["py.typed"],
|
||||
},
|
||||
platforms=['MacOS X', 'Posix'],
|
||||
test_suite='test',
|
||||
platforms=["MacOS X", "Posix"],
|
||||
test_suite="test",
|
||||
classifiers=[
|
||||
'Intended Audience :: Developers',
|
||||
'License :: OSI Approved :: Apache Software License',
|
||||
'Operating System :: MacOS :: MacOS X',
|
||||
'Operating System :: POSIX',
|
||||
'Programming Language :: Python',
|
||||
'Programming Language :: Python :: 3.7',
|
||||
'Programming Language :: Python :: 3.8',
|
||||
'Programming Language :: Python :: 3.9',
|
||||
'Programming Language :: Python :: 3.10',
|
||||
'Topic :: Software Development :: Libraries :: Python Modules'
|
||||
"Intended Audience :: Developers",
|
||||
"License :: OSI Approved :: Apache Software License",
|
||||
"Operating System :: MacOS :: MacOS X",
|
||||
"Operating System :: POSIX",
|
||||
"Programming Language :: Python",
|
||||
"Programming Language :: Python :: 3.7",
|
||||
"Programming Language :: Python :: 3.8",
|
||||
"Programming Language :: Python :: 3.9",
|
||||
"Programming Language :: Python :: 3.10",
|
||||
"Topic :: Software Development :: Libraries :: Python Modules",
|
||||
],
|
||||
)
|
||||
|
|
68
test/test.py
68
test/test.py
|
@ -1,15 +1,23 @@
|
|||
#!/usr/bin/env python
|
||||
|
||||
import os, sys, unittest, logging, base64, io, json
|
||||
import base64
|
||||
import io
|
||||
import json
|
||||
import logging
|
||||
import os
|
||||
import sys
|
||||
import unittest
|
||||
|
||||
import http_sfv
|
||||
import requests
|
||||
from requests.adapters import HTTPAdapter
|
||||
|
||||
sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
|
||||
sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), "..")))
|
||||
|
||||
from requests_http_signature import algorithms, HTTPSignatureAuth # noqa: E402
|
||||
from http_message_signatures import HTTPMessageSigner, InvalidSignature # noqa: E402
|
||||
from http_message_signatures import HTTPMessageSigner # noqa: E402
|
||||
from http_message_signatures import InvalidSignature # noqa: E402
|
||||
|
||||
from requests_http_signature import HTTPSignatureAuth, algorithms # noqa: E402
|
||||
|
||||
logging.basicConfig(level="DEBUG")
|
||||
|
||||
|
@ -24,8 +32,10 @@ class TestAdapter(HTTPAdapter):
|
|||
self.client_auth = auth
|
||||
|
||||
def send(self, request, *args, **kwargs):
|
||||
verify_args = dict(signature_algorithm=self.client_auth.signer.signature_algorithm,
|
||||
key_resolver=self.client_auth.signer.key_resolver)
|
||||
verify_args = dict(
|
||||
signature_algorithm=self.client_auth.signer.signature_algorithm,
|
||||
key_resolver=self.client_auth.signer.key_resolver,
|
||||
)
|
||||
HTTPSignatureAuth.verify(request, **verify_args)
|
||||
if request.body is not None:
|
||||
request.body = request.body[::-1]
|
||||
|
@ -41,14 +51,18 @@ class TestAdapter(HTTPAdapter):
|
|||
response.headers["Received-Signature-Input"] = request.headers["Signature-Input"]
|
||||
response.headers["Received-Signature"] = request.headers["Signature"]
|
||||
response.raw = io.BytesIO(json.dumps({}).encode())
|
||||
signer = HTTPMessageSigner(signature_algorithm=self.client_auth.signer.signature_algorithm,
|
||||
key_resolver=self.client_auth.signer.key_resolver)
|
||||
signer = HTTPMessageSigner(
|
||||
signature_algorithm=self.client_auth.signer.signature_algorithm,
|
||||
key_resolver=self.client_auth.signer.key_resolver,
|
||||
)
|
||||
hasher = HTTPSignatureAuth._content_digest_hashers["sha-256"]
|
||||
digest = hasher(response.raw.getvalue()).digest()
|
||||
response.headers["Content-Digest"] = str(http_sfv.Dictionary({"sha-256": digest}))
|
||||
signer.sign(response,
|
||||
key_id=default_keyid,
|
||||
covered_component_ids=("@method", "@authority", "content-digest", "@target-uri"))
|
||||
signer.sign(
|
||||
response,
|
||||
key_id=default_keyid,
|
||||
covered_component_ids=("@method", "@authority", "content-digest", "@target-uri"),
|
||||
)
|
||||
return response
|
||||
|
||||
|
||||
|
@ -65,7 +79,7 @@ class TestRequestsHTTPSignature(unittest.TestCase):
|
|||
self.session.mount("https://", TestAdapter(self.auth))
|
||||
|
||||
def test_basic_statements(self):
|
||||
url = 'http://example.com/path?query#fragment'
|
||||
url = "http://example.com/path?query#fragment"
|
||||
self.session.get(url, auth=self.auth)
|
||||
self.auth.signer.key_resolver.resolve_public_key = lambda k: b"abc"
|
||||
with self.assertRaises(InvalidSignature):
|
||||
|
@ -89,46 +103,44 @@ class TestRequestsHTTPSignature(unittest.TestCase):
|
|||
HTTPSignatureAuth.verify(res, **verify_args)
|
||||
|
||||
def test_auto_cover_authorization_header(self):
|
||||
url = 'http://example.com/path?query#fragment'
|
||||
url = "http://example.com/path?query#fragment"
|
||||
res = self.session.get(url, auth=self.auth, headers={"Authorization": "Bearer 12345"})
|
||||
self.assertIn('"authorization"', res.headers["Received-Signature-Input"])
|
||||
|
||||
def test_b21(self):
|
||||
url = 'https://example.com/foo?param=Value&Pet=dog'
|
||||
url = "https://example.com/foo?param=Value&Pet=dog"
|
||||
self.session.post(
|
||||
url,
|
||||
json={"hello": "world"},
|
||||
headers={
|
||||
"Date": "Tue, 20 Apr 2021 02:07:55 GMT",
|
||||
"Content-Digest": ("sha-512=:WZDPaVn/7XgHaAy8pmojAkGWoRx2UFChF41A2svX+TaPm+"
|
||||
"AbwAgBWnrIiYllu7BNNyealdVLvRwEmTHWXvJwew==:")
|
||||
"Content-Digest": (
|
||||
"sha-512=:WZDPaVn/7XgHaAy8pmojAkGWoRx2UFChF41A2svX+TaPm+"
|
||||
"AbwAgBWnrIiYllu7BNNyealdVLvRwEmTHWXvJwew==:"
|
||||
),
|
||||
},
|
||||
auth=self.auth
|
||||
auth=self.auth,
|
||||
)
|
||||
|
||||
@unittest.skip("TODO")
|
||||
def test_rsa(self):
|
||||
from cryptography.hazmat.backends import default_backend
|
||||
from cryptography.hazmat.primitives.asymmetric import rsa
|
||||
from cryptography.hazmat.primitives import serialization
|
||||
private_key = rsa.generate_private_key(
|
||||
public_exponent=65537,
|
||||
key_size=2048,
|
||||
backend=default_backend()
|
||||
)
|
||||
from cryptography.hazmat.primitives.asymmetric import rsa
|
||||
|
||||
private_key = rsa.generate_private_key(public_exponent=65537, key_size=2048, backend=default_backend())
|
||||
private_key_pem = private_key.private_bytes(
|
||||
encoding=serialization.Encoding.PEM,
|
||||
format=serialization.PrivateFormat.PKCS8,
|
||||
encryption_algorithm=serialization.BestAvailableEncryption(passphrase)
|
||||
encryption_algorithm=serialization.BestAvailableEncryption(passphrase),
|
||||
)
|
||||
public_key_pem = private_key.public_key().public_bytes(
|
||||
encoding=serialization.Encoding.PEM,
|
||||
format=serialization.PublicFormat.SubjectPublicKeyInfo
|
||||
encoding=serialization.Encoding.PEM, format=serialization.PublicFormat.SubjectPublicKeyInfo
|
||||
)
|
||||
url = 'http://example.com/path?query#fragment'
|
||||
url = "http://example.com/path?query#fragment"
|
||||
auth = HTTPSignatureAuth(algorithm="rsa-sha256", key=private_key_pem, key_id="sekret", passphrase=passphrase)
|
||||
self.session.get(url, auth=auth, headers=dict(pubkey=base64.b64encode(public_key_pem)))
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
|
|
Loading…
Reference in a new issue