Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Record webhook payload and which webhooks were fired with it #93

Open
xgp opened this issue Jan 17, 2025 · 0 comments
Open

Record webhook payload and which webhooks were fired with it #93

xgp opened this issue Jan 17, 2025 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@xgp
Copy link
Member

xgp commented Jan 17, 2025

  1. Add webhook_event table that has an compound fk to "type" (admin|event) and event_entity.id or admin_event_entity.id. Would require us to write a custom EventStoreProvider that gets the stored UUID and puts it somewhere (session attributes?) where we can get it.
  2. Add a webhook_sent table that associates webhook_event_entity with webhook and sent_at, retries, final_status
  • Do I need to do this in keycloak-events, or can it all happen here?
  • Do I need to plug into the methods that remove/clear events? No, just set on delete cascade.
  • Only activate this by a config flag set on the WebhookSenderEventListenerProviderFactory

Looks like the event ID is added to the Event and AdminEvent objects.

Started in #92

@xgp xgp self-assigned this Jan 17, 2025
@xgp xgp added the enhancement New feature or request label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant