Skip to content

Commit 5df11e2

Browse files
committed
new
Signed-off-by: TIANHE <TIANHE@GMAIL.COM>
1 parent 528be54 commit 5df11e2

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

backend_api_python/app/routes/auth.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -379,9 +379,14 @@ def login_with_code():
379379
(user['id'],)
380380
)
381381
db.commit()
382+
affected = cur.rowcount
382383
cur.close()
384+
if affected == 0:
385+
logger.error(f"Failed to update last_login_at: no rows affected for user_id={user['id']}")
386+
else:
387+
logger.info(f"Updated last_login_at for user_id={user['id']}")
383388
except Exception as e:
384-
logger.warning(f"Failed to update last_login_at: {e}")
389+
logger.error(f"Failed to update last_login_at for user_id={user.get('id')}: {e}")
385390

386391
# Log login
387392
security.log_security_event('login_via_code', user['id'], ip_address, user_agent)

backend_api_python/app/services/user_service.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,9 +170,14 @@ def authenticate(self, username: str, password: str) -> Optional[Dict[str, Any]]
170170
(user['id'],)
171171
)
172172
db.commit()
173+
affected = cur.rowcount
173174
cur.close()
175+
if affected == 0:
176+
logger.error(f"Failed to update last_login_at: no rows affected for user_id={user['id']}")
177+
else:
178+
logger.info(f"Updated last_login_at for user_id={user['id']}")
174179
except Exception as e:
175-
logger.warning(f"Failed to update last_login_at: {e}")
180+
logger.error(f"Failed to update last_login_at for user_id={user.get('id')}: {e}")
176181

177182
# Remove password_hash from return value
178183
user.pop('password_hash', None)

0 commit comments

Comments
 (0)