Skip to content

Commit 4793eaf

Browse files
Fix #2672: Pass db_session to user delete method
- Pass required db_session parameter to current_user.delete() - Add try-except block for proper exception handling - Store username before deletion for accurate flash message - Add error logging for debugging failed deletions Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com> Signed-off-by: guptapratykshh <pratykshgupta9999@gmail.com>
1 parent 946abed commit 4793eaf

1 file changed

Lines changed: 12 additions & 4 deletions

File tree

augur/api/view/routes.py

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -198,10 +198,18 @@ def authorize_user():
198198
@app.route('/account/delete')
199199
@login_required
200200
def user_delete():
201-
if current_user.delete()[0]:
202-
flash(f"Account {current_user.login_name} successfully removed")
203-
logout_user()
204-
else:
201+
try:
202+
username = current_user.login_name
203+
result = current_user.delete(db_session)
204+
205+
if result[0]:
206+
flash(f"Account {username} successfully removed")
207+
logout_user()
208+
else:
209+
logger.error(f"Failed to delete account {username}: {result[1]}")
210+
flash("An error occurred removing the account")
211+
except Exception as e:
212+
logger.error(f"Exception occurred while deleting account: {e}")
205213
flash("An error occurred removing the account")
206214

207215
return redirect(url_for("root"))

0 commit comments

Comments
 (0)