feat(AWS HTTP API): Use the default API Gateway timeout of 30s (#11223)

Author: CorentinDoue

docs/providers/aws/events/http-api.md

@@ -73,7 +73,7 @@ functions:
 
 ### Endpoints timeout
 
-Framework ensures that function timeout setting (which defaults to 6 seconds) is respected in HTTP API endpoint configuration. Still note that maximum possible timeout for an endpoint is 29 seconds. Ensure to keep function timeout below that. Otherwise you may observe successful lambda invocations reported with `503` status code.
+The default and maximal API Gateway timeout is used: 30s. Ensure to keep function timeout below 29s. Otherwise, you may observe successful lambda invocations reported with `503` status code.
 
 ### CORS Setup
 

lib/plugins/aws/package/compile/events/http-api.js

@@ -611,11 +611,6 @@ Object.defineProperties(
               'for a successful lambda invocation.'
           );
         }
-        // Ensure endpoint has slightly larger timeout than a function,
-        // It's a margin needed for some side processing time on AWS side.
-        // Otherwise there's a risk of observing 503 status for successfully resolved invocation
-        // (which just fit function timeout setting)
-        routeTargetData.timeout = Math.min(functionTimeout + 0.5, 30);
       }
     }),
     compileIntegration: d(function (routeTargetData) {
@@ -630,9 +625,6 @@ Object.defineProperties(
         IntegrationUri: resolveTargetConfig(routeTargetData),
         PayloadFormatVersion: funcHttpApi.payload || providerHttpApi.payload || '2.0',
       };
-      if (routeTargetData.timeout) {
-        properties.TimeoutInMillis = Math.round(routeTargetData.timeout * 1000);
-      }
       this.cfTemplate.Resources[
         this.provider.naming.getHttpApiIntegrationLogicalId(routeTargetData.functionName)
       ] = {

test/unit/lib/plugins/aws/package/compile/events/http-api.test.js

@@ -129,19 +129,9 @@ describe('lib/plugins/aws/package/compile/events/httpApi.test.js', () => {
       expect(resource.Properties.RouteKey).to.equal(routeKey);
     });
 
-    it('should ensure higher timeout than function default value', () => {
+    it('should let the default api gateway timeout by not setting the TimeoutInMillis property', () => {
       const resource = cfResources[naming.getHttpApiIntegrationLogicalId('foo')];
-      expect(resource.Properties.TimeoutInMillis).to.equal(6500);
-    });
-
-    it('should provide 0.5s time margin to custom function integration timeout', () => {
-      const resource = cfResources[naming.getHttpApiIntegrationLogicalId('customTimeout')];
-      expect(resource.Properties.TimeoutInMillis).to.equal(29500);
-    });
-
-    it('should limit function maximum integration timeout to 30s', () => {
-      const resource = cfResources[naming.getHttpApiIntegrationLogicalId('maxTimeout')];
-      expect(resource.Properties.TimeoutInMillis).to.equal(30000);
+      expect(resource.Properties.TimeoutInMillis).to.be.undefined;
     });
 
     it('should configure lambda permissions', () => {