-
Notifications
You must be signed in to change notification settings - Fork 3
fix(audit): rename audit events #1612
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
Conversation
Reviewer's GuideThis PR refactors audit event generation by renaming events to descriptive messages that include resource names and initiating users, adds filters to ignore internal system service accounts, refines conditions under which events are logged, and adjusts server logging levels. Class diagram for updated audit event classesclassDiagram
class VMAccess {
+IsMatched() bool
+Fill() error
eventLog: VMEventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes VM name, connection stage, subresource, and user
-- IsMatched() now ignores most system service accounts
}
class IntegrityCheckVM {
+IsMatched() bool
+Fill() error
eventLog: IntegrityCheckEventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes VM name
-- IsMatched() now ignores most system service accounts
}
class VMOPControl {
+IsMatched() bool
+Fill() error
eventLog: VMEventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes VM name and user for each operation
-- IsMatched() now ignores most system service accounts
}
class VMControl {
+Fill() error
eventLog: VMEventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes VM name for OS stop/restart, and abnormal kill includes user
-- Some system actions are now ignored
}
class VMManage {
+IsMatched() bool
+Fill() error
eventLog: VMEventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes VM name and user for create/update/delete
-- IsMatched() now ignores most system service accounts
}
class ModuleComponentControl {
+IsMatched() bool
+Fill() error
eventLog: EventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes component name and user for create/delete
-- IsMatched() now ignores most system service accounts
}
class Forbid {
+IsMatched() bool
+Fill() error
eventLog: EventLog
event: AuditEvent
-- Updated Fill():
-- eventLog.Name now includes user, verb, and resource in forbidden operation
-- IsMatched() now ignores most system service accounts
}
class ModuleControl {
+IsMatched() bool
eventLog: EventLog
event: AuditEvent
-- IsMatched() now ignores most system service accounts
}
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description Improving audit events names. Also add ignoring system service acconts. Signed-off-by: Daniil Antoshin <[email protected]>
d90ee3b to
f9d1365
Compare
Description
Improving audit events names. Also add ignoring system service acconts.
Checklist
Changelog entries