Log more admin actions

(Valentin Churavy) #1

I just noted that a few admin/moderator actions are not audited.
As an example Lock/Unlock user to Trust Level isn’t, but probably should be.

(Joshua Rosenfeld) #2

While we’re on the subject…

Blocks by the system user don’t have any context:

Delete post has no context (self-deleted in these cases):

Delete user reveals email address:

Also occurs with revoke email.

3 separate logs for same event (spam topic/user):

(Jeff Atwood) #3

Adding context should be a priority, @neil can you add to your list.

(Neil Lalonde) #4

Yeah we should definitely try to include context whenever we add something in those logs. Added to my list.

(Neil Lalonde) #5

I handled all the things mentioned in this topic.

  • log lock/unlock trust level
  • log activate/deactivate user
  • context added for deleted post because author deleted
  • stop displaying email addresses of deleted users. The screened emails list will show the emails of spammers/bad people.
  • stop displaying email address for revoked email (can be found on user record as usual)

Determine why 'system' user blocked a human user?
(Joshua Rosenfeld) #6

Were you able to handle this too?

(Neil Lalonde) #7

Yes the context will include something like “blocked by staff” and “too many flags”, from the reason for the block.

Determine why 'system' user blocked a human user?
(Erlend Sogge Heggen) #8

While we’re at it, I’ll take the liberty of bumping this:

(Jeff Atwood) #10

Can you confirm we are good here @neil and @erlend_sh?

(Erlend Sogge Heggen) #11

I’m still seeing some system deletions with empty descriptions, which is quite confusing:

(Jeff Atwood) #12

Those look like email staged users to me, so system is perhaps cleaning up old staged users, can you confirm @zogstrip?

(Régis Hanol) #13

Confirmed :heavy_check_mark: and fixed :wink:

(Jeff Atwood) #14