Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User activity audit logging #4073

Merged
merged 37 commits into from
Oct 31, 2022
Merged

User activity audit logging #4073

merged 37 commits into from
Oct 31, 2022

Conversation

lcampbell2
Copy link
Collaborator

@lcampbell2 lcampbell2 commented Oct 24, 2022

User activity within organizations is now being recorded and made available to admins within their organization view, as well as to super admins in an encompassing view.

Logs show information involving the following actions:

  • Create
  • Add
  • Update
  • Remove
  • Delete

and the following resource types:

  • Organizations
  • Users
  • Domains

The logs are displayed in a dynamic table with search, sorting, and new filtering by resource and actions.
A reason is now required for users to remove domain from their org, selectable from a dropdown.

closes #3995

api/src/audit-logs/objects/audit-log.js Outdated Show resolved Hide resolved
api/src/audit-logs/objects/initiated-by.js Outdated Show resolved Hide resolved
Comment on lines +8 to +12
resource: {
type: GraphQLString,
description: 'Name of the targeted resource.',
resolve: ({ resource }) => resource,
},
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think having the id for the resource as well as the name would be good

api/src/enums/log-order-field.js Outdated Show resolved Hide resolved
api/src/audit-logs/objects/target-resource.js Outdated Show resolved Hide resolved
api/src/audit-logs/objects/audit-log.js Outdated Show resolved Hide resolved
api/src/audit-logs/objects/audit-log.js Outdated Show resolved Hide resolved
api/src/audit-logs/objects/initiated-by.js Outdated Show resolved Hide resolved
api/src/audit-logs/queries/find-audit-logs.js Outdated Show resolved Hide resolved
api/src/audit-logs/queries/find-audit-logs.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@FestiveKyle FestiveKyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍 👍

@lcampbell2 lcampbell2 merged commit 1bc9ce7 into master Oct 31, 2022
@lcampbell2 lcampbell2 deleted the user-activity-audit-logging branch October 31, 2022 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User Activity Audit Logs
2 participants