Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Transaction logging doesn't contain request ID #9893

Closed
erikjohnston opened this issue Apr 27, 2021 · 1 comment · Fixed by #9895
Closed

Transaction logging doesn't contain request ID #9893

erikjohnston opened this issue Apr 27, 2021 · 1 comment · Fixed by #9895
Labels
S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@erikjohnston
Copy link
Member

It instead logs with runWithConnection.
@erikjohnston erikjohnston added S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. labels Apr 27, 2021
@clokep
Copy link
Member

clokep commented Apr 27, 2021

This is probably a regression from #9654 or one of the follow-ups (#9857 or #9829).

@clokep clokep mentioned this issue Apr 27, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use the parent's logging context name for runWithConnection. matrix-org/synapse
2 participants
@clokep @erikjohnston