Skip to content

Commit bbcc090

Browse files
authored
Merge pull request #3738 from OpenNeuroOrg/enable-x-forwarded-for-sentry
fix(server): Log x-forwarded-for header in traces
2 parents e923106 + 200292f commit bbcc090

File tree

1 file changed

+13
-1
lines changed
  • packages/openneuro-server/src/libs/authentication

1 file changed

+13
-1
lines changed

packages/openneuro-server/src/libs/authentication/jwt.ts

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import passport from "passport"
22
import refresh from "passport-oauth2-refresh"
33
import jwt from "jsonwebtoken"
4+
import * as Sentry from "@sentry/node"
45
import { decrypt } from "./crypto"
56
import User from "../../models/user"
67
import config from "../../config"
@@ -187,7 +188,18 @@ export const authenticate = (req, res, next) => {
187188
}
188189
}
189190
passport.authenticate("jwt", { session: false }, (err, user) => {
190-
req.login(user, { session: false }, () => next())
191+
req.login(user, { session: false }, () => {
192+
if (user) {
193+
Sentry.setUser({
194+
id: user.id,
195+
ip_address: req.headers["x-forwarded-for"] as string,
196+
})
197+
}
198+
Sentry.setContext("request_headers", {
199+
"x-forwarded-for": req.headers["x-forwarded-for"],
200+
})
201+
next()
202+
})
191203
})(req, res, next)
192204
}
193205
authenticateAsync()

0 commit comments

Comments
 (0)