Skip to content

Commit

Permalink
fix: undefined IP in staging
Browse files Browse the repository at this point in the history
  • Loading branch information
conradbekondo committed Jan 14, 2025
1 parent 7f125c1 commit 4356abb
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions server/handlers/auth.mts
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,8 @@ export function handleGoogleOauthCallback({ failureRedirect }: { failureRedirect

export async function handleUserSignIn(req: Request, res: Response) {
logger.info('signing in user');
const ip = String(req.header('client-ip'));
logger.info('request headers', req.headers);
const ip = Netlify?.context?.ip ?? req.ip ?? String(req.header('client-ip'));

try {
const { success, data } = UserSchema.safeParse(req.user)
Expand All @@ -132,6 +133,7 @@ export async function handleUserSignIn(req: Request, res: Response) {
logger.info('generating refresh token', { email: user.email });
const refreshToken = randomBytes(16).toString('hex');
const db = useUsersDb();
logger.info('updating database with tokens');
const { accessTokenId, refreshTokenId } = await db.transaction(async t => {
const [{ accessTokenId }] = await t.insert(users.accessTokens).values({
ip,
Expand All @@ -151,7 +153,6 @@ export async function handleUserSignIn(req: Request, res: Response) {
});

logger.info('generated token pair');

const signedAccessToken = sign({
email: user.email,
sub: user.id,
Expand All @@ -161,12 +162,14 @@ export async function handleUserSignIn(req: Request, res: Response) {
tokenId: accessTokenId
}, String(process.env['JWT_SECRET']), { expiresIn: process.env['JWT_LIFETIME'] })

logger.info('signed new access token');
const signedRefreshToken = sign({
value: refreshToken,
tokenId: refreshTokenId
}, String(process.env['JWT_SECRET']), {
expiresIn: process.env['REFRESH_TOKEN_LIFETIME']
});
logger.info('signed new refresh token');

logger.info('Signed in user', { email: user.email });
res.redirect(`/auth/oauth2/callback?access=${signedAccessToken}&refresh=${signedRefreshToken}`)
Expand Down

0 comments on commit 4356abb

Please sign in to comment.