Skip to content

Commit 8b69f88

Browse files
committed
hotfix: created a variable that contains the original endpoint
1 parent 446c636 commit 8b69f88

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

utils/azure_users.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ def get_user(self, user_id) -> AzureUser:
9999
assert 200 == response.status_code
100100
return self.to_azure_user(response.json())
101101

102-
def users(self) -> List[AzureUser]:
102+
def users(self) -> List[AzureUser]: # import pdb; pdb.set_trace()
103103
role_fields_params = ','.join(
104104
[field_name for field_name, _ in ROLE_FIELD_VALUES.values()]
105105
)
@@ -108,15 +108,16 @@ def users(self) -> List[AzureUser]:
108108
role_fields_params=role_fields_params,
109109
)
110110

111+
final_endpoint = endpoint
111112
exists_users = True
112113
users = []
113114
valid_users = []
114115
skip_token_attribute = '&$skiptoken='
115116

116117
while exists_users:
117118
response = requests.get(
118-
endpoint, auth=BearerAuth(self.access_token)
119-
)
119+
final_endpoint, auth=BearerAuth(self.access_token)
120+
)
120121
json_response = response.json()
121122
assert 200 == response.status_code
122123
assert 'value' in json_response
@@ -131,8 +132,9 @@ def users(self) -> List[AzureUser]:
131132
request_token = remaining_users_link.split(
132133
skip_token_attribute
133134
)[1]
134-
endpoint = endpoint + skip_token_attribute + request_token
135-
135+
final_endpoint = endpoint + skip_token_attribute + request_token
136+
137+
136138
for user in users:
137139
user_emails = user['otherMails']
138140
if(len(user_emails) != 0 and user_emails[0].split('@')[1] == 'ioet.com'):

0 commit comments

Comments
 (0)