fix: dockerfile expose is not overwritten
This commit is contained in:
parent
b3c8c881b7
commit
aefdc76805
@ -45,6 +45,9 @@ public function submit()
|
||||
$environment = $project->load(['environments'])->environments->where('name', $this->parameters['environment_name'])->first();
|
||||
|
||||
$port = get_port_from_dockerfile($this->dockerfile);
|
||||
if (!$port) {
|
||||
$port = 80;
|
||||
}
|
||||
$application = Application::create([
|
||||
'name' => 'dockerfile-' . new Cuid2(7),
|
||||
'repository_project_id' => 0,
|
||||
|
@ -113,7 +113,7 @@ function generateApplicationContainerName(Application $application, $pull_reques
|
||||
return $application->uuid . '-' . $now;
|
||||
}
|
||||
}
|
||||
function get_port_from_dockerfile($dockerfile): int
|
||||
function get_port_from_dockerfile($dockerfile): int|null
|
||||
{
|
||||
$dockerfile_array = explode("\n", $dockerfile);
|
||||
$found_exposed_port = null;
|
||||
@ -127,7 +127,7 @@ function get_port_from_dockerfile($dockerfile): int
|
||||
if ($found_exposed_port) {
|
||||
return (int)$found_exposed_port->value();
|
||||
}
|
||||
return 80;
|
||||
return null;
|
||||
}
|
||||
|
||||
function defaultLabels($id, $name, $pull_request_id = 0, string $type = 'application', $subType = null, $subId = null)
|
||||
|
Loading…
Reference in New Issue
Block a user