Malin/domnitor
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Improved error logging for Domain refresh

Browse Source
This commit is contained in:
Hosteroid
2025-10-21 15:16:56 +03:00
parent ceeb2cfae4
commit 716fdcffe6
2 changed files with 70 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
app/Controllers/DebugController.php
View File

@@ -22,6 +22,15 @@ class DebugController extends Controller
return; return;
} }
// Log debug tool usage
$logger = new \App\Services\Logger('debug');
$logger->info('WHOIS debug tool accessed', [
'domain' => $domain,
'user_id' => \Core\Auth::id(),
'ip_address' => $_SERVER['REMOTE_ADDR'] ?? 'unknown',
'user_agent' => $_SERVER['HTTP_USER_AGENT'] ?? 'unknown'
]);
// Get TLD // Get TLD
$parts = explode('.', $domain); $parts = explode('.', $domain);
$tld = $parts[count($parts) - 1]; $tld = $parts[count($parts) - 1];
@@ -313,6 +322,19 @@ class DebugController extends Controller
// Get parsed info using WhoisService // Get parsed info using WhoisService
$info = $whoisService->getDomainInfo($domain); $info = $whoisService->getDomainInfo($domain);
// Log debug results
$logger->info('WHOIS debug completed', [
'domain' => $domain,
'tld' => $tld,
'server' => $server,
'rdap_succeeded' => $rdapSucceeded,
'whois_fallback_used' => !$rdapSucceeded,
'parsed_status' => $info['status'] ?? 'unknown',
'parsed_registrar' => $info['registrar'] ?? 'unknown',
'parsed_expiration' => $info['expiration_date'] ?? 'unknown',
'user_id' => \Core\Auth::id()
]);
$this->view('debug/whois', [ $this->view('debug/whois', [
'domain' => $domain, 'domain' => $domain,
'server' => $server, 'server' => $server,

48
app/Controllers/DomainController.php
View File

@@ -373,10 +373,25 @@ class DomainController extends Controller
return; return;
} }
// Log domain refresh start
$logger = new \App\Services\Logger();
$logger->info('Domain refresh started', [
'domain_id' => $id,
'domain_name' => $domain['domain_name'],
'user_id' => \Core\Auth::id(),
'ip_address' => $_SERVER['REMOTE_ADDR'] ?? 'unknown'
]);
// Get fresh WHOIS information // Get fresh WHOIS information
$whoisData = $this->whoisService->getDomainInfo($domain['domain_name']); $whoisData = $this->whoisService->getDomainInfo($domain['domain_name']);
if (!$whoisData) { if (!$whoisData) {
$logger->error('Domain refresh failed - WHOIS data not retrieved', [
'domain_id' => $id,
'domain_name' => $domain['domain_name'],
'user_id' => \Core\Auth::id()
]);
$_SESSION['error'] = 'Could not retrieve WHOIS information'; $_SESSION['error'] = 'Could not retrieve WHOIS information';
// Check if we came from view page // Check if we came from view page
$referer = $_SERVER['HTTP_REFERER'] ?? ''; $referer = $_SERVER['HTTP_REFERER'] ?? '';
@@ -401,6 +416,16 @@ class DomainController extends Controller
'whois_data' => json_encode($whoisData) 'whois_data' => json_encode($whoisData)
]); ]);
// Log successful domain refresh
$logger->info('Domain refresh completed successfully', [
'domain_id' => $id,
'domain_name' => $domain['domain_name'],
'new_status' => $status,
'registrar' => $whoisData['registrar'],
'expiration_date' => $whoisData['expiration_date'],
'user_id' => \Core\Auth::id()
]);
$_SESSION['success'] = 'Domain information refreshed'; $_SESSION['success'] = 'Domain information refreshed';
// Check if we came from view page or list page // Check if we came from view page or list page
@@ -649,6 +674,15 @@ class DomainController extends Controller
$settingModel = new \App\Models\Setting(); $settingModel = new \App\Models\Setting();
$isolationMode = $settingModel->getValue('user_isolation_mode', 'shared'); $isolationMode = $settingModel->getValue('user_isolation_mode', 'shared');
// Log bulk refresh start
$logger = new \App\Services\Logger();
$logger->info('Bulk domain refresh started', [
'user_id' => $userId,
'domain_count' => count($domainIds),
'isolation_mode' => $isolationMode,
'ip_address' => $_SERVER['REMOTE_ADDR'] ?? 'unknown'
]);
$refreshed = 0; $refreshed = 0;
$failed = 0; $failed = 0;
@@ -664,6 +698,11 @@ class DomainController extends Controller
$whoisData = $this->whoisService->getDomainInfo($domain['domain_name']); $whoisData = $this->whoisService->getDomainInfo($domain['domain_name']);
if (!$whoisData) { if (!$whoisData) {
$logger->warning('Bulk refresh failed for domain - WHOIS data not retrieved', [
'domain_id' => $id,
'domain_name' => $domain['domain_name'] ?? 'unknown',
'user_id' => $userId
]);
$failed++; $failed++;
continue; continue;
} }
@@ -684,6 +723,15 @@ class DomainController extends Controller
$refreshed++; $refreshed++;
} }
// Log bulk refresh completion
$logger->info('Bulk domain refresh completed', [
'user_id' => $userId,
'total_domains' => count($domainIds),
'refreshed' => $refreshed,
'failed' => $failed,
'success_rate' => count($domainIds) > 0 ? round(($refreshed / count($domainIds)) * 100, 2) . '%' : '0%'
]);
$_SESSION['success'] = "Refreshed $refreshed domain(s)" . ($failed > 0 ? ", $failed failed" : ''); $_SESSION['success'] = "Refreshed $refreshed domain(s)" . ($failed > 0 ? ", $failed failed" : '');
$this->redirect('/domains'); $this->redirect('/domains');
} }