Update: 2026-05-06 17:13:24

This commit is contained in:
Hamza-Ayed
2026-05-06 17:13:24 +03:00
parent 019bff7e37
commit 3d4e636fbe
2 changed files with 94 additions and 74 deletions

View File

@@ -17,81 +17,94 @@ use App\Middleware\RateLimitMiddleware;
// Rate limit: 3 OTP requests per minute per IP // Rate limit: 3 OTP requests per minute per IP
RateLimitMiddleware::check(3, 60); RateLimitMiddleware::check(3, 60);
$data = Security::sanitize(input()); try {
$data = Security::sanitize(input());
// 1. Validate // 1. Validate
$errors = Validator::validate($data, [ $errors = Validator::validate($data, [
'phone' => 'required', 'phone' => 'required',
]); ]);
if ($errors) { if ($errors) {
json_error('رقم الهاتف مطلوب', 422, $errors); json_error('رقم الهاتف مطلوب', 422, $errors);
} }
$phone = preg_replace('/[^0-9+]/', '', $data['phone']); $phone = preg_replace('/[^0-9+]/', '', $data['phone']);
$phoneHash = hash('sha256', $phone); $phoneHash = hash('sha256', $phone);
// 2. Find user by phone hash // 2. Find user by phone hash OR plain phone (Support both schemas)
$db = Database::getInstance(); $db = Database::getInstance();
$stmt = $db->prepare("SELECT id, tenant_id, name, is_active FROM users WHERE phone_hash = ? LIMIT 1");
$stmt->execute([$phoneHash]);
$user = $stmt->fetch();
if (!$user) { // First, try to find by phone_hash. If it fails, we'll catch it.
try {
$stmt = $db->prepare("SELECT id, tenant_id, name, is_active FROM users WHERE phone_hash = ? LIMIT 1");
$stmt->execute([$phoneHash]);
$user = $stmt->fetch();
} catch (\PDOException $e) {
// Fallback to searching by plain phone if phone_hash column doesn't exist
$stmt = $db->prepare("SELECT id, tenant_id, name, is_active FROM users WHERE phone = ? LIMIT 1");
$stmt->execute([$phone]);
$user = $stmt->fetch();
}
if (!$user) {
// Don't reveal if phone exists — generic message // Don't reveal if phone exists — generic message
json_success(null, 'إذا كان الرقم مسجلاً، سيتم إرسال رمز التحقق'); json_success(null, 'إذا كان الرقم مسجلاً، سيتم إرسال رمز التحقق');
exit; exit;
} }
if (!$user['is_active']) { if (!$user['is_active']) {
json_error('الحساب معطّل. تواصل مع المسؤول.', 403); json_error('الحساب معطّل. تواصل مع المسؤول.', 403);
} }
// 3. Generate OTP (6 digits) // 3. Generate OTP (6 digits)
$otp = str_pad((string)random_int(100000, 999999), 6, '0', STR_PAD_LEFT); $otp = str_pad((string)random_int(100000, 999999), 6, '0', STR_PAD_LEFT);
$otpHash = password_hash($otp, PASSWORD_DEFAULT); $otpHash = password_hash($otp, PASSWORD_DEFAULT);
$expiresAt = date('Y-m-d H:i:s', time() + 300); // 5 minutes $expiresAt = date('Y-m-d H:i:s', time() + 300); // 5 minutes
// 4. Store OTP in database (or Redis if available) // 4. Store OTP in database (or Redis if available)
// Using a simple approach: store in a cache file per phone $cacheDir = STORAGE_PATH . '/cache/otp';
$cacheDir = STORAGE_PATH . '/cache/otp'; if (!is_dir($cacheDir)) {
if (!is_dir($cacheDir)) {
mkdir($cacheDir, 0755, true); mkdir($cacheDir, 0755, true);
} }
$otpData = [ $otpData = [
'hash' => $otpHash, 'hash' => $otpHash,
'user_id' => $user['id'], 'user_id' => $user['id'],
'attempts' => 0, 'attempts' => 0,
'max_attempts' => 5, 'max_attempts' => 5,
'expires_at' => time() + 300, 'expires_at' => time() + 300,
'created_at' => time(), 'created_at' => time(),
]; ];
$fp = fopen($cacheDir . '/otp_' . $phoneHash . '.json', 'w'); $fp = fopen($cacheDir . '/otp_' . $phoneHash . '.json', 'w');
if ($fp) { if ($fp) {
flock($fp, LOCK_EX); flock($fp, LOCK_EX);
fwrite($fp, json_encode($otpData)); fwrite($fp, json_encode($otpData));
flock($fp, LOCK_UN); flock($fp, LOCK_UN);
fclose($fp); fclose($fp);
} }
// 5. Send OTP via WhatsApp Proxy // 5. Send OTP via WhatsApp Proxy
$whatsappService = new \App\Services\WhatsAppProxyService(); $whatsappService = new \App\Services\WhatsAppProxyService();
$message = "رمز التحقق لتطبيق مُصادَق:\n*{$otp}*\n\nصالح لمدة 5 دقائق."; $message = "رمز التحقق لتطبيق مُصادَق:\n*{$otp}*\n\nصالح لمدة 5 دقائق.";
$result = $whatsappService->sendMessage($phone, $message); $result = $whatsappService->sendMessage($phone, $message);
if (!$result['success']) { if (!$result['success']) {
error_log("ERROR: Failed to send OTP WhatsApp to phone: {$phone}"); error_log("ERROR: Failed to send OTP WhatsApp to phone: {$phone}");
json_error('عذراً، فشل في إرسال رمز التحقق. الرجاء التأكد من صحة رقم الواتساب الخاص بك والمحاولة مرة أخرى.', 500, ['whatsapp_debug' => $result]); json_error('عذراً، فشل في إرسال رمز التحقق. الرجاء التأكد من صحة رقم الواتساب الخاص بك والمحاولة مرة أخرى.', 500, ['whatsapp_debug' => $result]);
} }
// Log for development (REMOVE IN PRODUCTION!) // Log for development (REMOVE IN PRODUCTION!)
if (env('APP_DEBUG', 'false') === 'true') { if (env('APP_DEBUG', 'false') === 'true') {
error_log("DEV OTP for {$phone}: {$otp}"); error_log("DEV OTP for {$phone}: {$otp}");
}
json_success(['whatsapp_debug' => $result], 'إذا كان الرقم مسجلاً، سيتم إرسال رمز التحقق عبر واتساب');
} catch (\Exception $e) {
json_error('Internal Server Error: ' . $e->getMessage(), 500);
} }
json_success(['whatsapp_debug' => $result], 'إذا كان الرقم مسجلاً، سيتم إرسال رمز التحقق عبر واتساب');

View File

@@ -18,6 +18,10 @@ class AuthController extends GetxController {
Future<void> requestOtp(String phoneNumber) async { Future<void> requestOtp(String phoneNumber) async {
try { try {
if (phoneNumber.trim().isEmpty) {
AppSnackbar.showError('خطأ', 'الرجاء إدخال رقم الهاتف أولاً');
return;
}
isLoading.value = true; isLoading.value = true;
phone.value = phoneNumber; phone.value = phoneNumber;
@@ -31,8 +35,11 @@ class AuthController extends GetxController {
Get.toNamed(AppRoutes.OTP_VERIFY); Get.toNamed(AppRoutes.OTP_VERIFY);
} }
} on DioException catch (e, stackTrace) { } on DioException catch (e, stackTrace) {
AppLogger.error('OTP Request Failed', e.response?.data, stackTrace); String errorMessage = 'فشل الاتصال بالخادم';
AppSnackbar.showError('خطأ', e.response?.data['message'] ?? 'فشل الاتصال بالخادم'); if (e.response?.data != null && e.response?.data is Map) {
errorMessage = e.response?.data['message'] ?? errorMessage;
}
AppSnackbar.showError('خطأ', errorMessage);
} finally { } finally {
isLoading.value = false; isLoading.value = false;
} }