Skip to content

Commit c8ddcdf

Browse files
authored
Merge pull request #36 from Mastercard/oauth_openapi_5.2.1
Changes to make it compatible with enc lib and OpenAPI Generator 5.2.1
2 parents b73f096 + 9b5e20e commit c8ddcdf

File tree

2 files changed

+17
-6
lines changed

2 files changed

+17
-6
lines changed

oauth1/signer_interceptor.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,9 @@ def request_function(*args, **kwargs): # pragma: no cover
6060

6161
in_headers["Authorization"] = auth_header
6262

63-
return func(*args, **kwargs)
63+
res = func(*args, **kwargs)
64+
65+
return res
6466

6567
request_function.__oauth__ = True
6668
return request_function
@@ -75,7 +77,8 @@ def add_signer_layer(api_client, key_file, key_password, consumer_key):
7577
"""Create and load configuration. Decorate APIClient.request with header signing"""
7678

7779
api_signer = SignerInterceptor(key_file, key_password, consumer_key)
78-
api_client.request = api_signer.oauth_signing(api_client.request)
80+
81+
api_client.rest_client.request = api_signer.oauth_signing(api_client.rest_client.request)
7982

8083

8184
@deprecated(version='1.1.3', reason="Use get_signer_layer(api_client) instead")
@@ -84,4 +87,4 @@ def get_signing_layer(self, api_client):
8487

8588

8689
def get_signer_layer(api_client):
87-
return api_client.request
90+
return api_client.rest_client.request

tests/test_interceptor.py

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,19 @@ def test_add_interceptor(self):
4040
key_password = "Password1"
4141
consumer_key = 'dummy'
4242

43-
signing_layer1 = get_signer_layer(requests)
44-
add_signer_layer(requests, key_file, key_password, consumer_key)
45-
signing_layer2 = get_signer_layer(requests)
43+
signer_request = MockApiRestClient(requests)
44+
45+
signing_layer1 = get_signer_layer(signer_request)
46+
add_signer_layer(signer_request, key_file, key_password, consumer_key)
47+
signing_layer2 = get_signer_layer(signer_request)
4648
self.assertNotEqual(signing_layer1, signing_layer2)
4749

4850

51+
class MockApiRestClient(object):
52+
def __init__(self, request):
53+
self.request = request
54+
self.rest_client = request
55+
56+
4957
if __name__ == '__main__':
5058
unittest.main()

0 commit comments

Comments
 (0)