rosslh/coolify
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix restart initiated duplicate and restore activity logs

Browse source 
- Add restartInitiated flag to prevent duplicate "Proxy restart initiated" messages
- Restore ProxyStatusChangedUI dispatch with activityId in RestartProxyJob
- This allows the UI to open the activity monitor and show logs during restart
- Simplified restart message (removed redundant "Monitor progress" text)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Andras Bacsai 2025-12-03 16:11:56 +01:00
parent b00d8902f4
commit c42fb81347
2 changed files with 17 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/Jobs/RestartProxyJob.php
View file

@ -46,9 +46,11 @@ public function handle()
// listener that handles UI updates and Traefik version checks // listener that handles UI updates and Traefik version checks
$activity = StartProxy::run($this->server, force: true, restarting: true); $activity = StartProxy::run($this->server, force: true, restarting: true);
// Store activity ID for reference // Store activity ID and dispatch event with it so UI can open activity monitor
if ($activity && is_object($activity)) { if ($activity && is_object($activity)) {
$this->activity_id = $activity->id; $this->activity_id = $activity->id;
// Dispatch event with activity ID so the UI can show logs
ProxyStatusChangedUI::dispatch($this->server->team_id, $this->activity_id);
} }
} catch (\Throwable $e) { } catch (\Throwable $e) {

15
app/Livewire/Server/Navbar.php
View file

@ -30,6 +30,8 @@ class Navbar extends Component
public ?string $lastNotifiedStatus = null; public ?string $lastNotifiedStatus = null;
public bool $restartInitiated = false;
public function getListeners() public function getListeners()
{ {
$teamId = auth()->user()->currentTeam()->id; $teamId = auth()->user()->currentTeam()->id;
@ -65,11 +67,22 @@ public function restart()
try { try {
$this->authorize('manageProxy', $this->server); $this->authorize('manageProxy', $this->server);
// Prevent duplicate restart messages (e.g., from double-click or re-render)
if ($this->restartInitiated) {
return;
}
$this->restartInitiated = true;
// Always use background job for all servers // Always use background job for all servers
RestartProxyJob::dispatch($this->server); RestartProxyJob::dispatch($this->server);
$this->dispatch('info', 'Proxy restart initiated. Monitor progress in activity logs.'); $this->dispatch('info', 'Proxy restart initiated.');
// Reset the flag after a short delay to allow future restarts
$this->restartInitiated = false;
} catch (\Throwable $e) { } catch (\Throwable $e) {
$this->restartInitiated = false;
return handleError($e, $this); return handleError($e, $this);
} }
} }