Skip to content

Commit 316f605

Browse files
Fix MemberManagement Access Type Safety
1 parent b7a2468 commit 316f605

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

conditional/blueprints/member_management.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -290,8 +290,10 @@ def member_management_getuserinfo(uid):
290290
if not ldap_is_eval_director(user_name) and not ldap_is_financial_director(user_name):
291291
return "must be eval or financial director", 403
292292

293-
acct = FreshmanAccount.query.filter(
294-
FreshmanAccount.id == uid).first()
293+
acct = None
294+
if uid.isnumeric():
295+
acct = FreshmanAccount.query.filter(
296+
FreshmanAccount.id == uid).first()
295297

296298
# missed hm
297299
def get_hm_date(hm_id):

0 commit comments

Comments
 (0)