more security improvements

server/api/auth/login.post.ts

@@ -3,14 +3,24 @@ import { setAuthCookie, generateSessionToken } from '~/server/utils/auth'
 import bcrypt from 'bcrypt'
 
 export default defineEventHandler(async (event) => {
-  // Get client IP for rate limiting
+  // Get real client IP from proxy headers (prioritize x-real-ip for NPM)
-  const clientIp = getRequestIP(event) || 'unknown'
+  const xRealIp = getHeader(event, 'x-real-ip')
+  const xForwardedFor = getHeader(event, 'x-forwarded-for')
+  const cfConnectingIp = getHeader(event, 'cf-connecting-ip')
   
-  // Log IP for verification (helps ensure correct public IP is captured)
+  // Use x-real-ip first (set by NPM), then x-forwarded-for, then cf-connecting-ip, then fallback
+  const clientIp = xRealIp || 
+                   (xForwardedFor ? xForwardedFor.split(',')[0].trim() : null) ||
+                   cfConnectingIp ||
+                   getRequestIP(event) || 
+                   'unknown'
+  
+  // Log IP for verification
   console.log(`[LOGIN ATTEMPT] IP: ${clientIp}, Headers:`, {
-    'x-forwarded-for': getHeader(event, 'x-forwarded-for'),
+    'x-forwarded-for': xForwardedFor,
-    'x-real-ip': getHeader(event, 'x-real-ip'),
+    'x-real-ip': xRealIp,
-    'cf-connecting-ip': getHeader(event, 'cf-connecting-ip')
+    'cf-connecting-ip': cfConnectingIp,
+    'getRequestIP': getRequestIP(event)
   })
   
   // Check rate limit: 5 attempts per 15 minutes

server/api/auth/register.post.ts

@@ -2,14 +2,24 @@ import { createUser, getUserByUsername, getUserByEmail, checkRateLimit, createSe
 import { setAuthCookie, generateSessionToken } from '~/server/utils/auth'
 
 export default defineEventHandler(async (event) => {
-  // Get client IP for rate limiting
+  // Get real client IP from proxy headers (prioritize x-real-ip for NPM)
-  const clientIp = getRequestIP(event) || 'unknown'
+  const xRealIp = getHeader(event, 'x-real-ip')
+  const xForwardedFor = getHeader(event, 'x-forwarded-for')
+  const cfConnectingIp = getHeader(event, 'cf-connecting-ip')
+  
+  // Use x-real-ip first (set by NPM), then x-forwarded-for, then cf-connecting-ip, then fallback
+  const clientIp = xRealIp || 
+                   (xForwardedFor ? xForwardedFor.split(',')[0].trim() : null) ||
+                   cfConnectingIp ||
+                   getRequestIP(event) || 
+                   'unknown'
   
   // Log IP for verification
   console.log(`[REGISTER ATTEMPT] IP: ${clientIp}, Headers:`, {
-    'x-forwarded-for': getHeader(event, 'x-forwarded-for'),
+    'x-forwarded-for': xForwardedFor,
-    'x-real-ip': getHeader(event, 'x-real-ip'),
+    'x-real-ip': xRealIp,
-    'cf-connecting-ip': getHeader(event, 'cf-connecting-ip')
+    'cf-connecting-ip': cfConnectingIp,
+    'getRequestIP': getRequestIP(event)
   })
   
   // Check rate limit: 3 attempts per hour