Browser client (websocket) logins not tracking user activity

Bug #1818153 reported by Bill Erickson on 2019-02-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
High
Unassigned
3.1
Undecided
Unassigned

Bug Description

Evergreen 3.2

Logins to the staff client are not represented as rows in actor.usr_activity. This is a result of missing config.usr_activity_type rows for websocket logins.

Proposal is to add 4 new activity types:

ewhat login, ehow ws-translator-v1
ewhat login, ehow ws-translator-v2
ewhat login
ewhat verivy

The latter 2 ensure that new ingress options are represented going forward even when new activity types are not specifically added for them.

Bill Erickson (berick) wrote :

And ws-translator-v* 'verify' entries.

Bill Erickson (berick) wrote :
Changed in evergreen:
milestone: none → 3.2.4
assignee: Bill Erickson (berick) → nobody
Bill Erickson (berick) wrote :

To test, apply the SQL changes, then login to the browser client. There should now be an entry in actor.usr_activity for the logged in user of either etype of 25 or 26, depending on which websocket gateway is active.

Changed in evergreen:
milestone: 3.2.4 → 3.2.5
Michele Morgan (mmorgan) on 2019-03-05
Changed in evergreen:
assignee: nobody → Michele Morgan (mmorgan)
Michele Morgan (mmorgan) wrote :

I applied the sql changes and am now seeing rows added to actor.usr_activity for web client logins. My signoff branch is at:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mmorgan/lp1818153_websocket_activity_types_signoff

Changed in evergreen:
status: New → Confirmed
tags: added: pullrequest signedoff
Bill Erickson (berick) on 2019-03-05
Changed in evergreen:
assignee: Michele Morgan (mmorgan) → Bill Erickson (berick)
Bill Erickson (berick) wrote :

Thanks, Michelle. Merged to 3.1 and above.

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Bill Erickson (berick) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers