Skip to content

Conversation

@jasoncabot
Copy link
Contributor

@jasoncabot jasoncabot commented Apr 29, 2025

Overview

This gives the ability to query a particular account for a number of audit logs

image

Notes

I kept everything in the apps folder (rather than moving to common) for now to help keep things localised, but happy to move stuff around in the future.

TODO

  • Double check query schema
  • Add auth scope for auditlogs:read

@Maximo-Guk
Copy link
Member

Maximo-Guk commented Apr 30, 2025

This was merged in !100!
( I think once this PR get's rebased off the main branch fork in the fork, it should automatically reflect as merged )

  • Made a couple changes to wrangler.jsonc to include WAE bindings
  • Referenced our observability scriptName in UserDetails durable object since we want to share the same DO namespace ( Ideally we probably want some other dedicated worker to host this durable object at some point )
  • Renamed OAUTH_KV_AUDITLOGS to OAUTH_KV ( since we reference OAUTH_KV in our auth code )
  • Rebased + added devHandler that we're now using in all our servers ( no need to hackily pass your apiToken in props now when developing! ), and moved to using apiHandler so that we get SSE + Streamable HTTP support

@Maximo-Guk Maximo-Guk merged commit c6647ad into cloudflare:main Apr 30, 2025
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.

2 participants