Skip to content

Circuit-breaker events + audit-log pagination - #342

Merged
Idaonoli merged 3 commits into
stellar-compliance-kit:mainfrom
deeejaymu-afk:drips-danjuma
Sep 3, 2026
Merged

Circuit-breaker events + audit-log pagination#342
Idaonoli merged 3 commits into
stellar-compliance-kit:mainfrom
deeejaymu-afk:drips-danjuma

Conversation

@deeejaymu-afk

@deeejaymu-afk deeejaymu-afk commented Aug 30, 2026

Copy link
Copy Markdown

Summary

  • Emit Frozen/Unfrozen contract events from circuit-breaker's freeze/unfreeze, following the repo's shared event conventions (admin address as topic), with EVENTS.md and indexer decoder updates.
  • Add a bounded, paginated list_entries(env, start, limit) -> Vec<LogEntry> view to audit-log, with a dedicated PageTooLarge error when limit exceeds MAX_PAGE_SIZE, plus unit tests covering multi-page pagination and the overflow error.

Notes

  • Add integration test: allowlist-token writes to audit-log on add/remove #208 and Add integration test: denylist-gate writes to audit-log on add/remove #209 (audit-log wiring for allowlist-token/denylist-gate) were not completed in this PR: contracts/allowlist-token/src/lib.rs and contracts/denylist-gate/src/lib.rs currently have pre-existing issues unrelated to audit-log (duplicate function definitions, undefined symbols such as Metadata, PendingAdmin, caller, Paused/Unpaused/GatePaused events, ComplianceOfficer/BatchTooLarge/SignerSet, and missing imports). Wiring in audit-log calls on top of that would not produce reviewable or verifiable changes; commented on both issues explaining this, and this PR closes them as no further action is possible until the base files are repaired in a follow-up.

Closes #210
Closes #207
Closes #208
Closes #209

@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

@luhrhenz is attempting to deploy a commit to the idaonoli-2655's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@deeejaymu-afk Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Idaonoli
Idaonoli merged commit fc4fa3b into stellar-compliance-kit:main Sep 3, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants