Skip to content

[ENHENCEMENT] Ease finding a Dead Letter event by eventId - #3132

Open
chibenwa wants to merge 1 commit into
apache:masterfrom
chibenwa:webadmin-event-dead-letter-event-id
Open

[ENHENCEMENT] Ease finding a Dead Letter event by eventId#3132
chibenwa wants to merge 1 commit into
apache:masterfrom
chibenwa:webadmin-event-dead-letter-event-id

Conversation

@chibenwa

Copy link
Copy Markdown
Contributor

No description provided.

@chibenwa chibenwa linked an issue Aug 23, 2026 that may be closed by this pull request
Arsnael pushed a commit to linagora/twake-mail-admin that referenced this pull request Aug 24, 2026
Per reviewer suggestion (cf apache/james-project#3132), the search by event
ID now accepts an optional listener group that narrows the query to a single
group, which is significantly cheaper when the group is known.

- Frontend: optional 'Group' input in the search subsection, gated on its own
  permission so it only renders when the proxy grants the group-scoped call.
- API client: appends &group=... only when a group is provided.
- validation.md + profile-editor inventory: declare the group-scoped endpoint
  GET /events/deadLetter?eventId={eventId}&group={group} as MAY, alongside the
  existing basic search, so generated profiles grant both.

Also fixes the three CI failures introduced by the profile-editor merge:
- inventory + crosscheck now cover GET /events/deadLetter?eventId={eventId}
  (declared in validation.md but absent from the inventory).
- test_resolver.py mirrors the new event-ID query cases transcribed upstream
  in proxy-resolver.test.ts.
- checksums.sha256 refreshed for proxy-resolver.test.ts after re-reading the
  port against it (no resolver behaviour change needed).
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.

Webadmin: dead letter - search by event ID

3 participants