diff --git a/api/funkwhale_api/federation/factories.py b/api/funkwhale_api/federation/factories.py index 574560f0d..971c774a7 100644 --- a/api/funkwhale_api/federation/factories.py +++ b/api/funkwhale_api/federation/factories.py @@ -20,11 +20,10 @@ class SignatureAuthFactory(factory.Factory): algorithm = "rsa-sha256" key = factory.LazyFunction(lambda: keys.get_key_pair()[0]) key_id = factory.Faker("url") - use_auth_header = False headers = ["(request-target)", "user-agent", "host", "date", "accept"] class Meta: - model = requests_http_signature.HTTPSignatureAuth + model = requests_http_signature.HTTPSignatureHeaderAuth @registry.register(name="federation.SignedRequest") diff --git a/api/funkwhale_api/federation/signing.py b/api/funkwhale_api/federation/signing.py index b69c48668..17a0228a4 100644 --- a/api/funkwhale_api/federation/signing.py +++ b/api/funkwhale_api/federation/signing.py @@ -46,7 +46,7 @@ def verify(request, public_key): verify_date(date) try: return requests_http_signature.HTTPSignatureAuth.verify( - request, key_resolver=lambda **kwargs: public_key, use_auth_header=False + request, key_resolver=lambda **kwargs: public_key, scheme="Signature" ) except cryptography.exceptions.InvalidSignature: logger.warning( @@ -98,8 +98,7 @@ def verify_django(django_request, public_key): def get_auth(private_key, private_key_id): - return requests_http_signature.HTTPSignatureAuth( - use_auth_header=False, + return requests_http_signature.HTTPSignatureHeaderAuth( headers=["(request-target)", "user-agent", "host", "date"], algorithm="rsa-sha256", key=private_key.encode("utf-8"), diff --git a/api/requirements/base.txt b/api/requirements/base.txt index 07a7b0ade..4b8d6c688 100644 --- a/api/requirements/base.txt +++ b/api/requirements/base.txt @@ -45,9 +45,7 @@ uvicorn[standard]~=0.12.0 gunicorn~=20.0.0 cryptography~=2.9.0 -# requests-http-signature==0.0.3 -# clone until the branch is merged and released upstream -git+https://github.com/EliotBerriot/requests-http-signature.git@signature-header-support +requests-http-signature==0.2.0 django-cleanup~=5.0.0 requests~=2.24.0 pyOpenSSL~=19.1.0 diff --git a/changes/changelog.d/876.enhancement b/changes/changelog.d/876.enhancement new file mode 100644 index 000000000..d567c5e57 --- /dev/null +++ b/changes/changelog.d/876.enhancement @@ -0,0 +1 @@ +Replaced forked http-signature dependency with official package (#876)