Skip to content

Commit 102927f

Browse files
Merge pull request #26 from RIT-EVT/bug_fix/token_type
Bug fix/token type
2 parents 020a81f + 327b8e6 commit 102927f

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

TelemetrySite/server/tests/conftest.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,19 +78,19 @@ def setup_config_doc(mock_db):
7878
{
7979
"username": "test_user_valid",
8080
"password": "123".encode(),
81-
"auth_token": 0,
81+
"auth_token": "0",
8282
"auth_time": datetime.now()
8383
},
8484
{
8585
"username": "outdated_user",
8686
"password": "123".encode(),
87-
"auth_token": 1,
87+
"auth_token": "1",
8888
"auth_time": datetime.min
8989
},
9090
{
9191
"username": "duplicate_user",
9292
"password": "123".encode(),
93-
"auth_token": 2,
93+
"auth_token": "2",
9494
"auth_time": datetime.now()
9595
}
9696
])

TelemetrySite/server/utils.py

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ def authenticate_user(auth_token, db):
5959
# If it is then allow the operation to continue
6060
# Otherwise return an error
6161
auth_connection = db["users"]
62-
auth_token=convert_to_int(auth_token)
6362
return (auth_connection.find_one({"auth_token":auth_token})!=None)
6463

6564
def check_expired_tokens(auth_token, db):
@@ -74,7 +73,6 @@ def check_expired_tokens(auth_token, db):
7473
"""
7574

7675
auth_connection = db["users"]
77-
auth_token=convert_to_int(auth_token)
7876
auth_time = auth_connection.find_one({"auth_token":auth_token})["auth_time"]
7977
# only allow an auth token to last a day
8078
days = (datetime.now()-auth_time).days
@@ -97,10 +95,4 @@ def update_expired_token(document_id, db):
9795

9896
auth_connection.update_one( {"_id": ObjectId(document_id)},
9997
{"$set": {"auth_token":new_auth_token, "auth_time":datetime.now()}})
100-
return new_auth_token
101-
102-
def convert_to_int(value):
103-
104-
if type(value) == str:
105-
return int(value)
106-
return value
98+
return new_auth_token

0 commit comments

Comments
 (0)