Skip to content

Commit 1bbd879

Browse files
Add v1 check to delete user
1 parent a05735e commit 1bbd879

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

authsignal/client.py

+6-2
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,7 @@ def get_user(self, user_id, redirect_url=None, path=None):
152152
def delete_user(self, user_id):
153153
_assert_non_empty_unicode(user_id, 'user_id')
154154

155-
user_id = urllib.parse.quote(user_id)
156-
path = f'{self.url}/v1/users/{user_id}'
155+
path = self._delete_user_url(user_id)
157156
headers = self._default_headers()
158157

159158
try:
@@ -291,6 +290,11 @@ def _validate_challenge_url(self):
291290
path = self._ensure_versioned_path(f'/validate')
292291
return f'{self.url}{path}'
293292

293+
def _delete_user_url(self, user_id):
294+
user_id = urllib.parse.quote(user_id)
295+
path = self._ensure_versioned_path(f'/users/{user_id}')
296+
return f'{self.url}{path}'
297+
294298
def _ensure_versioned_path(self, path):
295299
if not self.url.endswith(f'/{self.api_version}'):
296300
return f'/{self.api_version}{path}'

authsignal/client_tests.py

+1
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,7 @@ def test_delete_user_authenticator(self):
154154
self.authsignal_client = client.Client(api_key='test_api_key')
155155
user_id = 'test_user'
156156
user_authenticator_id = 'test_authenticator'
157+
157158
expected_url = f'{self.authsignal_client.url}/v1/users/{user_id}/authenticators/{user_authenticator_id}'
158159

159160
responses.add(responses.DELETE, expected_url, json={"success": True}, status=200)

0 commit comments

Comments
 (0)