paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-04 05:53:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

apple auth: Change logger to zulip.auth.apple from root logger.

Browse Source 
Replace default root logger with zulip.auth.apple for apple auth
in file zproject/backends.py and update the test cases
accordingly in file zerver/tests/test_auth_backends.py
This commit is contained in:
palash
2020-07-14 22:48:28 +05:30
committed by Tim Abbott
parent 04f08f45e7
commit 485d740fe8
2 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
zerver/tests/test_auth_backends.py
View File

@@ -2310,24 +2310,24 @@ class AppleAuthBackendNativeFlowTest(AppleAuthMixin, SocialAuthBase):
result = self.client_get(url, **headers)
self.assertEqual(result.status_code, 302)
with self.assertLogs(level='INFO') as info_log:
with self.assertLogs(self.logger_string, level='INFO') as info_log:
# Start Apple auth with mobile_flow_otp param. It should get saved into the session
# on SOCIAL_AUTH_SUBDOMAIN.
initiate_auth(mobile_flow_otp)
self.assertEqual(self.client.session['mobile_flow_otp'], mobile_flow_otp)
self.assertEqual(info_log.output, [
'INFO:root:/complete/apple/: Authentication failed: Token validation failed'
self.logger_output("/complete/apple/: Authentication failed: Token validation failed", "info")
])
with self.assertLogs(level='INFO') as info_log:
with self.assertLogs(self.logger_string, level='INFO') as info_log:
# Make a request without mobile_flow_otp param and verify the field doesn't persist
# in the session from the previous request.
initiate_auth()
self.assertEqual(self.client.session.get('mobile_flow_otp'), None)
self.assertEqual(info_log.output, [
'INFO:root:/complete/apple/: Authentication failed: Token validation failed'
self.logger_output("/complete/apple/: Authentication failed: Token validation failed", "info")
])
def test_id_token_with_invalid_aud_sent(self) -> None:
@@ -2337,9 +2337,12 @@ class AppleAuthBackendNativeFlowTest(AppleAuthMixin, SocialAuthBase):
id_token=self.generate_id_token(account_data_dict, audience='com.different.app'),
)
with self.apple_jwk_url_mock(), self.assertLogs(level='INFO') as m:
with self.apple_jwk_url_mock(), self.assertLogs(self.logger_string, level='INFO') as m:
result = self.client_get(url, **headers)
self.assertEqual(m.output, ["INFO:root:/complete/apple/: {}".format("Authentication failed: Token validation failed")])
self.assertEqual(m.output, [
self.logger_output("/complete/apple/: Authentication failed: Token validation failed", "info")
])
return result
def test_social_auth_desktop_success(self) -> None:

2
zproject/backends.py
View File

@@ -1673,7 +1673,7 @@ class AppleAuthBackend(SocialAuthMixin, AppleIdAuth):
# though AuthFailed would have been more correct.
#
# We have an open PR to python-social-auth to clean this up.
logging.info("/complete/apple/: %s", str(e))
self.logger.info("/complete/apple/: %s", str(e))
return None
@external_auth_method