Skip to content

Commit 9f5cd5e

Browse files
authored
chore: Remove use of legacy restore, which is now deprecated (#295)
* chore: Remove use of legacy restore, which is now deprecated * chore: Changes requested by amcconnell
1 parent b964296 commit 9f5cd5e

2 files changed

Lines changed: 2 additions & 10 deletions

File tree

duo_client/admin.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2040,8 +2040,8 @@ def update_settings(self,
20402040
helpdesk_bypass_expiration - <int:minutes>|0
20412041
helpdesk_message - <str:message|None>
20422042
helpdesk_can_send_enroll_email - True|False|None
2043-
reactivation_url - <str:url>|None
2044-
reactivation_integration_key - <str:url>|None
2043+
reactivation_url - Deprecated; ignored if specified.
2044+
reactivation_integration_key - Deprecated; ignored if specified.
20452045
security_checkup_enabled - True|False|None
20462046
user_managers_can_put_users_in_bypass - True|False|None
20472047
email_activity_notification_enabled = True|False|None
@@ -2113,10 +2113,6 @@ def update_settings(self,
21132113
if helpdesk_can_send_enroll_email is not None:
21142114
params['helpdesk_can_send_enroll_email'] = ('1' if
21152115
helpdesk_can_send_enroll_email else '0')
2116-
if reactivation_url is not None:
2117-
params['reactivation_url'] = reactivation_url
2118-
if reactivation_integration_key is not None:
2119-
params['reactivation_integration_key'] = reactivation_integration_key
21202116
if security_checkup_enabled is not None:
21212117
params['security_checkup_enabled'] = ('1' if
21222118
security_checkup_enabled else '0')

tests/admin/test_settings.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,6 @@ def test_update_settings(self):
3737
helpdesk_bypass_expiration=60,
3838
helpdesk_message="test_message",
3939
helpdesk_can_send_enroll_email=True,
40-
reactivation_url="https://www.example.com",
41-
reactivation_integration_key='DINTEGRATIONKEYTEST0',
4240
security_checkup_enabled=True,
4341
user_managers_can_put_users_in_bypass=False,
4442
email_activity_notification_enabled=True,
@@ -79,8 +77,6 @@ def test_update_settings(self):
7977
'helpdesk_bypass_expiration': '60',
8078
'helpdesk_message': 'test_message',
8179
'helpdesk_can_send_enroll_email': '1',
82-
'reactivation_url': 'https://www.example.com',
83-
'reactivation_integration_key': 'DINTEGRATIONKEYTEST0',
8480
'security_checkup_enabled': '1',
8581
'user_managers_can_put_users_in_bypass': '0',
8682
'email_activity_notification_enabled': '1',

0 commit comments

Comments
 (0)