Skip to content

Add accounting logs page to admin interface#545

Merged
J08nY merged 3 commits intopagefrom
copilot/add-accounting-logs-page
Feb 12, 2026
Merged

Add accounting logs page to admin interface#545
J08nY merged 3 commits intopagefrom
copilot/add-accounting-logs-page

Conversation

Copy link
Contributor

Copilot AI commented Dec 25, 2025

Plan: Add Accounting Logs Page to Admin Interface

  • Explore repository structure and understand existing admin interface
  • Review accounting collection schema and existing code
  • Review test structure for admin functionality
  • Add accounting logs route to admin views
  • Create accounting logs template with filtering UI
  • Add link to accounting logs in admin index page
  • Create test data for accounting logs
  • Write tests for the new accounting logs page
  • Validate filter parameters to prevent injection attacks
  • Run tests and verify functionality
  • Test with manual verification (if needed)
Original prompt

Add a page to the admin interface that shows accounting logs from the accounting collection. Allow to filter per-user and per-endpoint logs.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Comment on lines +283 to +286
if username_filter:
query["username"] = username_filter
if endpoint_filter:
query["endpoint"] = endpoint_filter
Copy link
Member

Choose a reason for hiding this comment

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

@copilot Validate the following filters as they come from the user and should not go straight to a databse query.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added validation to ensure both username_filter and endpoint_filter are strings or None before using them in the MongoDB query. This prevents potential injection attacks where a user could pass dictionary objects with MongoDB operators. Fixed in e225c66.

Copilot AI requested a review from J08nY January 23, 2026 11:22
@J08nY J08nY marked this pull request as ready for review February 12, 2026 12:06
@J08nY J08nY changed the title [WIP] Add accounting logs page to admin interface Add accounting logs page to admin interface Feb 12, 2026
@J08nY J08nY added enhancement New feature or request web Stuff related to the web at seccerts.org labels Feb 12, 2026
@J08nY J08nY force-pushed the copilot/add-accounting-logs-page branch from e225c66 to b021971 Compare February 12, 2026 12:23
@J08nY J08nY merged commit 04e712a into page Feb 12, 2026
3 checks passed
@J08nY J08nY deleted the copilot/add-accounting-logs-page branch February 12, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request web Stuff related to the web at seccerts.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants