Skip to content

Fix EventQueue ClientFromHandler to Inject ent client | Enrich River Job submission with Listener Info#2070

Merged
golanglemonade merged 2 commits intomainfrom
bug-fixgalalisteners
Feb 25, 2026
Merged

Fix EventQueue ClientFromHandler to Inject ent client | Enrich River Job submission with Listener Info#2070
golanglemonade merged 2 commits intomainfrom
bug-fixgalalisteners

Conversation

@matoszz
Copy link
Member

@matoszz matoszz commented Feb 25, 2026

gala's ContextManager.Restore reconstructs auth claims and log fields from the durable snapshot, but the ent client is a live runtime dependency that isn't serializable - so it's absent from the restored context. Any downstream code calling generated.FromContext(ctx) (e.g., ent interceptors doing FGA/module checks) would get nil and fail. By seeding the client in ClientFromHandler, all callers get an enriched context without needing per-handler boilerplate.

Additionally, adds better logging into trust center listeners, enriches the River job submission with additional metadata.

@matoszz matoszz requested a review from a team as a code owner February 25, 2026 19:46
@github-actions github-actions bot added the bug Something isn't working label Feb 25, 2026
@golanglemonade golanglemonade enabled auto-merge (squash) February 25, 2026 19:49
@sonarqubecloud
Copy link

@golanglemonade golanglemonade merged commit fde9703 into main Feb 25, 2026
17 checks passed
@golanglemonade golanglemonade deleted the bug-fixgalalisteners branch February 25, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants