RobotRock

Pull Request: Feature Implementation

21 jul 2026•a few seconds ago

Unassigned

This PR implements a new authentication feature with JWT tokens. Review this code submission and provide feedback

PR Metadata

Author	developer@example.com
Repository	backend-api
Branch	feature/jwt-auth
PR Number	#1234
Files Changed	5
Lines Added	234
Lines Removed	12
Breaking Changes	No

Your decision

Choose an action and submit to continue the workflow.

Approve Code

Request Changes

Pull Request: Feature Implementation

21 jul 2026•a few seconds ago

Unassigned

This PR implements a new authentication feature with JWT tokens. Review this code submission and provide feedback

/src/auth/jwt.ts

import jwt from 'jsonwebtoken';
 
export function generateToken(userId: string): string {
-  return jwt.sign({ userId }, process.env.JWT_SECRET, { expiresIn: '24h' });
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.sign(
+    { userId, iat: Math.floor(Date.now() / 1000) },
+    secret,
+    { expiresIn: '24h' }
+  );
}
 
export function verifyToken(token: string): { userId: string } {
-  return jwt.verify(token, process.env.JWT_SECRET) as { userId: string };
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.verify(token, secret) as { userId: string };
}

PR Metadata

Author	developer@example.com
Repository	backend-api
Branch	feature/jwt-auth
PR Number	#1234
Files Changed	5
Lines Added	234
Lines Removed	12
Breaking Changes	No

Your decision

Choose an action and submit to continue the workflow.

Approve Code

Request Changes

Pull Request: Feature Implementation

21 jul 2026•a few seconds ago

Unassigned

This PR implements a new authentication feature with JWT tokens. Review this code submission and provide feedback

/src/auth/jwt.ts

import jwt from 'jsonwebtoken';
 
export function generateToken(userId: string): string {
-  return jwt.sign({ userId }, process.env.JWT_SECRET, { expiresIn: '24h' });
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.sign(
+    { userId, iat: Math.floor(Date.now() / 1000) },
+    secret,
+    { expiresIn: '24h' }
+  );
}
 
export function verifyToken(token: string): { userId: string } {
-  return jwt.verify(token, process.env.JWT_SECRET) as { userId: string };
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.verify(token, secret) as { userId: string };
}

PR Metadata

Author	developer@example.com
Repository	backend-api
Branch	feature/jwt-auth
PR Number	#1234
Files Changed	5
Lines Added	234
Lines Removed	12
Breaking Changes	No

Your decision

Choose an action and submit to continue the workflow.

Approve Code

Request Changes

/src/auth/jwt.ts

import jwt from 'jsonwebtoken';
 
export function generateToken(userId: string): string {
-  return jwt.sign({ userId }, process.env.JWT_SECRET, { expiresIn: '24h' });
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.sign(
+    { userId, iat: Math.floor(Date.now() / 1000) },
+    secret,
+    { expiresIn: '24h' }
+  );
}
 
export function verifyToken(token: string): { userId: string } {
-  return jwt.verify(token, process.env.JWT_SECRET) as { userId: string };
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.verify(token, secret) as { userId: string };
}

/src/auth/jwt.ts

import jwt from 'jsonwebtoken';
 
export function generateToken(userId: string): string {
-  return jwt.sign({ userId }, process.env.JWT_SECRET, { expiresIn: '24h' });
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.sign(
+    { userId, iat: Math.floor(Date.now() / 1000) },
+    secret,
+    { expiresIn: '24h' }
+  );
}
 
export function verifyToken(token: string): { userId: string } {
-  return jwt.verify(token, process.env.JWT_SECRET) as { userId: string };
+  const secret = process.env.JWT_SECRET;
+  if (!secret) {
+    throw new Error('JWT_SECRET is not defined');
+  }
+  
+  return jwt.verify(token, secret) as { userId: string };
}