infrastructure/blender-id
0
4
Fork 0
Code Issues 11 Pull Requests Projects Releases Wiki Activity

User session tracking #93587

Merged
Oleg-Komarov merged 18 commits from user-session into main 2024-08-02 16:04:09 +02:00
Conversation 0 Commits 18 Files Changed 14 +4 -4
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 5177bf3b51 - Show all commits

2
bid_main/signals.py
View File

@ -38,7 +38,7 @@ def process_new_login(sender, request, user, **kwargs):
fields.update({"last_login_ip", "current_login_ip"})
if user.has_confirmed_email:
Oleg-Komarov marked this conversation as resolved
Anna Sirota commented 2024-08-02 11:30:13 +02:00
Review
Copy Link

should be a call of a task

should be a call of a task
bid_main.tasks.send_new_user_session(
bid_main.tasks.send_new_user_session_email(
user_pk=user.pk,
session_data={
'device': str(user_session.device or 'Unknown'),

2
bid_main/tasks.py
View File

@ -12,7 +12,7 @@ log = logging.getLogger(__name__)
@background(schedule={'action': TaskSchedule.RESCHEDULE_EXISTING})
def send_new_user_session(user_pk, session_data):
def send_new_user_session_email(user_pk, session_data):
user = User.objects.get(pk=user_pk)
log.info("sending a new user session email for account %s", user.pk)

4
bid_main/tests/test_user_sessions.py
View File

@ -50,8 +50,8 @@ class TestActiveSessions(TestCase):
class TestNewUserSessionEmail(TestCase):
@patch(
'bid_main.tasks.send_new_user_session',
new=bid_main.tasks.send_new_user_session.task_function,
'bid_main.tasks.send_new_user_session_email',
new=bid_main.tasks.send_new_user_session_email.task_function,
)
@patch(
'django.contrib.auth.base_user.AbstractBaseUser.check_password',