Merge branch 'master' into v3

Author: mnapoli

.gitignore

@@ -1,15 +1,9 @@
 /vendor
 /.idea
-/tests/Bridge/Symfony/var
-/tests/Bridge/Symfony/cache
-/tests/Bridge/Symfony/logs
 /.bref/
 /.phpcs-cache
 /composer.lock
-/.couscous/
 /website/node_modules/
-/website/template/output.css
-/.netlify/
 .DS_Store
 /.serverless/
 .phpunit.result.cache

LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2022 Matthieu Napoli
+Copyright (c) 2025 Matthieu Napoli
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

Makefile

@@ -1,11 +1,5 @@
 .EXPORT_ALL_VARIABLES:
 
-trigger_runtimes:
-	aws codepipeline start-pipeline-execution --name bref-php-binary
-
-runtime_build_status:
-	aws codepipeline get-pipeline-state --name=bref-php-binary | jq ".stageStates[1].latestExecution.status"
-
 layers.json:
 	php utils/layers.json/update.php
 
@@ -15,4 +9,4 @@ test-stack:
 preview:
 	cd website && make preview
 
-.PHONY: demo layers.json test-stack
+.PHONY: layers.json test-stack

composer.json

@@ -30,7 +30,7 @@
         "psr/http-message": "^1.0|^2.0",
         "psr/http-server-handler": "^1.0",
         "riverline/multipart-parser": "^2.1.2",
-        "symfony/process": "^4.4|^5.0|^6.0|^7.0"
+        "symfony/process": "^4.4|^5.0|^6.0|^7.0|^8.0"
     },
     "require-dev": {
         "async-aws/core": "^1.0",
@@ -42,8 +42,8 @@
         "guzzlehttp/guzzle": "^7.5",
         "phpstan/phpstan": "^1.10.26",
         "phpunit/phpunit": "^9.6.10",
-        "symfony/console": "^4.4|^5.0|^6.0|^7.0",
-        "symfony/yaml": "^4.4|^5.0|^6.0|^7.0"
+        "symfony/console": "^4.4|^5.0|^6.0|^7.0|^8.0",
+        "symfony/yaml": "^4.4|^5.0|^6.0|^7.0|^8.0"
     },
     "scripts": {
         "test": [

docs/laravel/caching.mdx

@@ -4,9 +4,9 @@ By default, the Bref bridge will move Laravel's storage and cache directories to
 
 However, the `/tmp` directory isn't shared across Lambda instances. If you Lambda function scales up or is redeployed, the cache will be empty in new instances.
 
-If you want the cache to be shared across all Lambda instances, for example if your application caches a lot of data or if you use it for locking mechanisms (like API rate limiting), you can instead use Redis or DynamoDB.
+If you want the cache to be shared across all Lambda instances, for example if your application caches a lot of data or if you use it for locking mechanisms (like API rate limiting), you can instead use the database, Redis, or DynamoDB.
 
-DynamoDB is the easiest to set up and is "pay per use". Redis is a bit more complex as it requires a VPC and managing instances, but offers slightly faster response times.
+If you are using a database already, using it as the cache driver is the simplest option. DynamoDB is a good alternative: fairly easy to set up, and "pay per use". Redis is a bit more complex as it requires a VPC and managing instances, but offers slightly faster response times.
 
 ## DynamoDB Cache
 

docs/setup/minimal-aws-policy.mdx

@@ -66,6 +66,7 @@ Feel free to submit a PR if you have any suggestions for improvements.
                 "iam:DeleteRolePolicy",
                 "iam:DetachRolePolicy",
                 "iam:GetRole",
+                "iam:TagRole",
                 "iam:PassRole",
                 "iam:PutRolePolicy",
                 "iot:CreateTopicRule",

src/Event/Http/HttpHandler.php

@@ -25,9 +25,9 @@ public function handle($event, Context $context): array
         $response = $this->handleRequest($httpEvent, $context);
 
         if ($httpEvent->isFormatV2()) {
-            return $response->toApiGatewayFormatV2();
+            return $response->toApiGatewayFormatV2($context->getAwsRequestId());
         }
 
-        return $response->toApiGatewayFormat($httpEvent->hasMultiHeader());
+        return $response->toApiGatewayFormat($httpEvent->hasMultiHeader(), $context->getAwsRequestId());
     }
 }

src/Event/Http/HttpResponse.php

@@ -21,7 +21,7 @@ public function __construct(string $body, array $headers = [], int $statusCode =
         $this->statusCode = $statusCode;
     }
 
-    public function toApiGatewayFormat(bool $multiHeaders = false): array
+    public function toApiGatewayFormat(bool $multiHeaders = false, ?string $awsRequestId = null): array
     {
         $base64Encoding = (bool) getenv('BREF_BINARY_RESPONSES');
 
@@ -38,6 +38,8 @@ public function toApiGatewayFormat(bool $multiHeaders = false): array
             }
         }
 
+        $this->checkHeadersSize($headers, $awsRequestId);
+
         // The headers must be a JSON object. If the PHP array is empty it is
         // serialized to `[]` (we want `{}`) so we force it to an empty object.
         $headers = empty($headers) ? new \stdClass : $headers;
@@ -58,7 +60,7 @@ public function toApiGatewayFormat(bool $multiHeaders = false): array
     /**
      * See https://docs.aws.amazon.com/apigateway/latest/developerguide/http-api-develop-integrations-lambda.html#http-api-develop-integrations-lambda.response
      */
-    public function toApiGatewayFormatV2(): array
+    public function toApiGatewayFormatV2(?string $awsRequestId = null): array
     {
         $base64Encoding = (bool) getenv('BREF_BINARY_RESPONSES');
 
@@ -76,6 +78,11 @@ public function toApiGatewayFormatV2(): array
             }
         }
 
+        $this->checkHeadersSize(array_merge(
+            $headers,
+            ['Set-Cookie' => $cookies], // include cookies in the size check
+        ), $awsRequestId);
+
         // The headers must be a JSON object. If the PHP array is empty it is
         // serialized to `[]` (we want `{}`) so we force it to an empty object.
         $headers = empty($headers) ? new \stdClass : $headers;
@@ -98,4 +105,31 @@ private function capitalizeHeaderName(string $name): string
         $name = ucwords($name);
         return str_replace(' ', '-', $name);
     }
+
+    /**
+     * API Gateway v1 and v2 have a headers total max size of 10 KB.
+     * ALB has a max size of 32 KB.
+     * It's hard to calculate the exact size of headers here, so we just
+     * estimate it roughly: if above 9.5 KB we log a warning.
+     *
+     * @param array<string|string[]> $headers
+     */
+    private function checkHeadersSize(array $headers, ?string $awsRequestId): void
+    {
+        $estimatedHeadersSize = 0;
+        foreach ($headers as $name => $values) {
+            $estimatedHeadersSize += strlen($name);
+            if (is_array($values)) {
+                foreach ($values as $value) {
+                    $estimatedHeadersSize += strlen($value);
+                }
+            } else {
+                $estimatedHeadersSize += strlen($values);
+            }
+        }
+
+        if ($estimatedHeadersSize > 9_500) {
+            echo "$awsRequestId\tWARNING\tThe total size of HTTP response headers is estimated to be above 10 KB, which is the API Gateway limit. If the limit is reached, the HTTP response will be a 500 error.\n";
+        }
+    }
 }