Skip to content

Conversation

@Jossec101
Copy link
Contributor

@Jossec101 Jossec101 commented Jan 16, 2026

Stacked PRs:


[GEN-1859] Audit logging system with enums and db migrations

  • 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

@Jossec101 Jossec101 marked this pull request as draft January 16, 2026 11:47
Jossec101 added a commit that referenced this pull request Jan 16, 2026
- 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
@Jossec101 Jossec101 force-pushed the Jossec101/stack/9 branch 4 times, most recently from 81ba785 to 2add3f4 Compare January 20, 2026 10:36
- 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
@Jossec101 Jossec101 marked this pull request as ready for review January 20, 2026 10:53
@Jossec101 Jossec101 requested review from RodriFS and imclvr January 20, 2026 10:53
Copy link
Contributor

@RodriFS RodriFS left a comment

Choose a reason for hiding this comment

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

flawless

@imclvr
Copy link

imclvr commented Jan 20, 2026

@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.

@Jossec101
Copy link
Contributor Author

@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.

@Jossec101 Jossec101 merged commit 822e8f4 into main Jan 22, 2026
1 check passed
@Jossec101 Jossec101 deleted the Jossec101/stack/9 branch January 22, 2026 11:10
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.

4 participants