这是indexloc提供的服务，不要输入任何密码
Skip to content

Attach request ID as tracing metadata #5456

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.

Sign up for GitHub

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

Merged
merged 25 commits into from
Jul 28, 2020
Merged

Attach request ID as tracing metadata #5456

Changes from all commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
ae373fe
Propagate the trace context to event triggers
paf31 Jul 16, 2020
13804bd
Handle missing trace and span IDs
paf31 Jul 17, 2020
41452cb
Store trace context as one LOCAL
paf31 Jul 17, 2020
cd68fb9
Add migrations
paf31 Jul 17, 2020
a070a3a
Merge branch 'master' into propagate-to-event-triggers
paf31 Jul 17, 2020
c58370d
Documentation
paf31 Jul 17, 2020
c282da1
Merge branch 'propagate-to-event-triggers' of github.com:paf31/graphq…
paf31 Jul 17, 2020
56dc03d
Include the request ID as trace metadata
paf31 Jul 17, 2020
a20e43a
Merge branch 'master' into propagate-to-event-triggers
paf31 Jul 20, 2020
0fcf2cd
Merge branch 'master' into propagate-to-event-triggers
paf31 Jul 21, 2020
71b88f3
changelog
paf31 Jul 21, 2020
f0bb23a
Fix warnings
paf31 Jul 21, 2020
08b7f93
Merge branch 'master' into propagate-to-event-triggers
paf31 Jul 21, 2020
44c2eec
Respond to code review suggestions
paf31 Jul 21, 2020
1ba565c
Merge branch 'propagate-to-event-triggers' of github.com:paf31/graphq…
paf31 Jul 21, 2020
9919c67
Respond to code review
paf31 Jul 22, 2020
e277454
Merge branch 'master' into propagate-to-event-triggers
paf31 Jul 22, 2020
c40ffa0
Merge branch 'master' into propagate-to-event-triggers
0x777 Jul 23, 2020
6cf7ce0
Undo changelog
paf31 Jul 23, 2020
7c6fede
Update CHANGELOG.md
paf31 Jul 23, 2020
f4e82ac
Merge remote-tracking branch 'paf31/propagate-to-event-triggers' into…
paf31 Jul 23, 2020
170c8db
Merge remote-tracking branch 'origin' into trace-request-id
paf31 Jul 23, 2020
86f1c6e
Typo
paf31 Jul 23, 2020
db0ba36
Merge branch 'master' into trace-request-id
paf31 Jul 28, 2020
48218cd
Merge branch 'master' into trace-request-id
paf31 Jul 28, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions server/src-lib/Hasura/Server/App.hs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,9 +275,13 @@ mkSpockAction serverCtx qErrEncoder qErrModifier apiHandler = do
tracingCtx
(fromString (B8.unpack pathInfo))

requestId <- getRequestId headers

requestId <- getRequestId headers
mapActionT runTraceT $ do
-- Add the request ID to the tracing metadata so that we
-- can correlate requests and traces
lift $ Tracing.attachMetadata [("request_id", unRequestId requestId)]

userInfoE <- fmap fst <$> lift (resolveUserInfo logger manager headers authMode)
userInfo <- either (logErrorAndResp Nothing requestId req (Left reqBody) False headers . qErrModifier)
return userInfoE
Expand Down