-
Notifications
You must be signed in to change notification settings - Fork 4
[GEN-1859] Audit logging system with enums and db migrations #465
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
39a56dd to
4dfb05e
Compare
4dfb05e to
733b9cf
Compare
- Add AuditLog model with tracking for user actions and system events - Define AuditActionType, AuditEventType, and AuditObjectType enums - Include Entity Framework configuration and database migrations - Support for user identification, IP tracking, and detailed event logging TO MERGE Implement AuditService with structured logging, HTTP context extraction, and database persistence for tracking user actions, system events, and security-related operations across the NodeGuard application. stack-info: PR: #465, branch: Jossec101/stack/9 [GEN-1859] Add HttpContext extension for retrieving client IP address and create Audit Trail page with filtering options stack-info: PR: #466, branch: Jossec101/stack/10
81ba785 to
2add3f4
Compare
2add3f4 to
d368ef4
Compare
- Add AuditLog model with tracking for user actions and system events - Define AuditActionType, AuditEventType, and AuditObjectType enums - Include Entity Framework configuration and database migrations - Support for user identification, IP tracking, and detailed event logging stack-info: PR: #465, branch: Jossec101/stack/9
d368ef4 to
326ed68
Compare
RodriFS
left a comment
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.
flawless
|
@Jossec101 one question, are we gonna use a separate DB instance for storing audit logs, or are we going to use same NG Postgres? I think that ideally this should be apart, so it could have its own RBAC, logging and in case main DB is compromised, this one is isolated and hopefully not affected. |
I dont think we need that complexity right now, the main use case is to capture logs via stdout and use a SIEM, the audit logs view + db is for nodeguard standalone as OSS project (easy audit trails) or for non technical users such as Finance Managers. |
Stacked PRs:
[GEN-1859] Audit logging system with enums and db migrations