dave/coolify
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix(api): enhance validation for docker_compose_raw in ApplicationsController

Browse Source 
- Add checks to ensure docker_compose_raw is provided and correctly base64 encoded.
- Implement detailed error responses for validation failures to improve user feedback.
This commit is contained in:
Andras Bacsai
2025-05-19 13:30:31 +02:00
parent 38d9471de1
commit cee9dfcf52
1 changed files with 28 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
app/Http/Controllers/Api/ApplicationsController.php
View File

@@ -1122,7 +1122,34 @@ class ApplicationsController extends Controller
$dockerComposeDomainsJson = collect(); $dockerComposeDomainsJson = collect();
if ($request->has('docker_compose_domains')) { if ($request->has('docker_compose_domains')) {
$yaml = Yaml::parse($application->docker_compose_raw); if (! $request->has('docker_compose_raw')) {
return response()->json([
'message' => 'Validation failed.',
'errors' => [
'docker_compose_raw' => 'The base64 encoded docker_compose_raw is required.',
],
], 422);
}
if (! isBase64Encoded($request->docker_compose_raw)) {
return response()->json([
'message' => 'Validation failed.',
'errors' => [
'docker_compose_raw' => 'The docker_compose_raw should be base64 encoded.',
],
], 422);
}
$dockerComposeRaw = base64_decode($request->docker_compose_raw);
if (mb_detect_encoding($dockerComposeRaw, 'ASCII', true) === false) {
return response()->json([
'message' => 'Validation failed.',
'errors' => [
'docker_compose_raw' => 'The docker_compose_raw should be base64 encoded.',
],
], 422);
}
$dockerComposeRaw = base64_decode($request->docker_compose_raw);
$yaml = Yaml::parse($dockerComposeRaw);
$services = data_get($yaml, 'services'); $services = data_get($yaml, 'services');
$dockerComposeDomains = collect($request->docker_compose_domains); $dockerComposeDomains = collect($request->docker_compose_domains);
if ($dockerComposeDomains->count() > 0) { if ($dockerComposeDomains->count() > 0) {