Skip to content

Add database-backed audit logs for mutations#298

Closed
zhangsiqiang519 wants to merge 1 commit into
BountyOnChain:mainfrom
zhangsiqiang519:codex/stellar-bounty-audit-log-198
Closed

Add database-backed audit logs for mutations#298
zhangsiqiang519 wants to merge 1 commit into
BountyOnChain:mainfrom
zhangsiqiang519:codex/stellar-bounty-audit-log-198

Conversation

@zhangsiqiang519

Copy link
Copy Markdown

Summary

  • add an AuditLog entity, migration, module, and AuditService
  • persist audit events for bounty create/update/delete/restore
  • persist audit events for submission create/approve/reject
  • scrub sensitive metadata keys before saving audit records
  • pass authenticated wallet address into bounty mutations for actor attribution
  • add focused unit coverage and changelog entry

Closes #198

Verification

  • git diff --check
  • npm run build --workspace=apps/backend
  • npm run lint --workspace=apps/backend
  • npm run test --workspace=apps/backend -- audit.service.spec.ts bounties.service.spec.ts submissions.service.spec.ts submissions.contract-errors.spec.ts

@GBOYEE

GBOYEE commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

I'd like to work on this issue. I have background in backend and will submit a PR ASAP.

@GBOYEE

GBOYEE commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

I'd like to work on this.

Approach:

  • I'll add the audit logging with proper schema and query support
  • Verify with existing tests + add new ones if needed

Estimated effort: ~1-2 hours. PR incoming shortly.
Closes #298

Copy link
Copy Markdown
Contributor

Thanks for this. Issue #198 is assigned to @barry01-hash — only the assigned contributor's PR is being merged here, so I'll hold off. Worth checking in on the issue.

Copy link
Copy Markdown
Contributor

Closing this — #198 is assigned to @barry01-hash. Thanks for putting this together!

@Xuccessor Xuccessor closed this Jun 23, 2026
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.

[FEATURE] No request/response logging for audit trail

3 participants