Skip to content

E
el-captain-manager
Commit d522d5bd authored by Mahmoud Aglan's avatar Mahmoud Aglan
Browse files
Options

Fix: enable SSL before forcing it (CapRover requires domain SSL first) 

Co-Authored-By: 's avatarClaude Opus 4.6 <noreply@anthropic.com>
parent eb5a517f
Show whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
app/Services/InstanceProvisionerService.php
View file @ d522d5bd
...@@ -137,7 +137,6 @@ private function deployApp(Instance $instance, string $dbPassword, string $admin ...@@ -137,7 +137,6 @@ private function deployApp(Instance $instance, string $dbPassword, string $admin
$this->caprover->configureAppFull($instance->app_name, [ $this->caprover->configureAppFull($instance->app_name, [
'containerHttpPort' => 80, 'containerHttpPort' => 80,
'instanceCount' => 1, 'instanceCount' => 1,
'forceSsl' => true,
'volumes' => [ 'volumes' => [
['containerPath' => '/var/www/html/storage/app', 'volumeName' => "{$instance->app_name}-storage"], ['containerPath' => '/var/www/html/storage/app', 'volumeName' => "{$instance->app_name}-storage"],
], ],
...@@ -153,10 +152,11 @@ private function deployApp(Instance $instance, string $dbPassword, string $admin ...@@ -153,10 +152,11 @@ private function deployApp(Instance $instance, string $dbPassword, string $admin
], ],
]); ]);
// Step 3: Enable SSL // Step 3: Enable SSL then force it
$progress('Enabling SSL...'); $progress('Enabling SSL...');
try { try {
$this->caprover->enableSsl($instance->app_name); $this->caprover->enableSsl($instance->app_name);
$this->caprover->forceSsl($instance->app_name);
} catch (\Throwable $e) { } catch (\Throwable $e) {
// Wildcard cert handles this — not critical // Wildcard cert handles this — not critical
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment