Aws cloudfront event schema

lib/plugins/aws/package/compile/events/cloudFront/index.js

@@ -27,9 +27,118 @@ class AwsCompileCloudFrontEvents {
       'before:remove:remove': this.logRemoveReminder.bind(this),
     };
 
-    // TODO: Complete schema, see https://github.com/serverless/serverless/issues/8025
+    const originObjectSchema = {
+      type: 'object',
+      properties: {
+        ConnectionAttempts: { type: 'integer', miminum: 1, maximum: 3 },
+        ConnectionTimeout: { type: 'integer', miminum: 1, maximum: 10 },
+        CustomOriginConfig: {
+          type: 'object',
+          properties: {
+            HTTPPort: { type: 'integer' },
+            HTTPSPort: { type: 'integer' },
+            OriginKeepaliveTimeout: { type: 'integer', miminum: 1, maximum: 60 },
+            OriginProtocolPolicy: {
+              type: 'string',
+              enum: ['http-only', 'match-viewer', 'https-only'],
+            },
+            OriginReadTimeout: { type: 'integer', miminum: 1, maximum: 60 },
+            OriginSSLProtocols: {
+              type: 'array',
+              items: { type: 'string', enum: ['SSLv3', 'TLSv1', 'TLSv1.1', 'TLSv1.2'] },
+            },
+          },
+          additionalProperties: false,
+          required: ['OriginProtocolPolicy'],
+        },
+        DomainName: { type: 'string' },
+        Id: { type: 'string' },
+        OriginCustomHeaders: {
+          type: 'array',
+          items: {
+            type: 'object',
+            properties: {
+              HeaderName: { type: 'string' },
+              HeaderValue: { type: 'string' },
+            },
+            additionalProperties: false,
+            required: ['HeaderName', 'HeaderValue'],
+          },
+        },
+        OriginPath: { type: 'string' },
+        S3OriginConfig: {
+          type: 'object',
+          properties: {
+            OriginAccessIdentity: {
+              type: 'string',
+              pattern: '^origin-access-identity/cloudfront/.+',
+            },
+          },
+          additionalProperties: false,
+        },
+      },
+      additionalProperties: false,
+      required: ['DomainName', 'Id'],
+      oneOf: [{ required: ['CustomOriginConfig'] }, { required: ['S3OriginConfig'] }],
+    };
+
+    const behaviorObjectSchema = {
+      type: 'object',
+      properties: {
+        AllowedMethods: {
+          oneOf: [
+            { type: 'array', items: [{ const: 'GET' }, { const: 'HEAD' }] },
+            { type: 'array', items: [{ const: 'GET' }, { const: 'HEAD' }, { const: 'OPTIONS' }] },
+            {
+              type: 'array',
+              items: [
+                { const: 'GET' },
+                { const: 'HEAD' },
+                { const: 'OPTIONS' },
+                { const: 'PUT' },
+                { const: 'PATCH' },
+                { const: 'POST' },
+                { const: 'DELETE' },
+              ],
+            },
+          ],
+        },
+        CachedMethods: {
+          oneOf: [
+            { type: 'array', items: [{ const: 'GET' }, { const: 'HEAD' }] },
+            { type: 'array', items: [{ const: 'GET' }, { const: 'HEAD' }, { const: 'OPTIONS' }] },
+          ],
+        },
+        CachePolicyId: { type: 'string' },
+        Compress: { type: 'boolean' },
+        FieldLevelEncryptionId: { type: 'string' },
+        OriginRequestPolicyId: { type: 'string' },
+        SmoothStreaming: { type: 'boolean' },
+        TrustedSigners: { type: 'array', items: { type: 'string' } },
+        ViewerProtocolPolicy: {
+          type: 'string',
+          enum: ['allow-all', 'redirect-to-https', 'https-only'],
+        },
+      },
+      additionalProperties: false,
+    };
+
     this.serverless.configSchemaHandler.defineFunctionEvent('aws', 'cloudFront', {
       type: 'object',
+      properties: {
+        eventType: {
+          type: 'string',
+          enum: ['viewer-request', 'origin-request', 'origin-response', 'viewer-response'],
+        },
+        pathPattern: { type: 'string', pattern: '^([A-Za-z0-9_.*$/~"\'@:+-]|&)+$' },
+        origin: {
+          oneOf: [{ type: 'string', format: 'uri' }, originObjectSchema],
+        },
+        isDefaultOrigin: { type: 'boolean' },
+        includeBody: { type: 'boolean' },
+        behavior: behaviorObjectSchema,
+      },
+      additionalProperties: false,
     });
   }
 
@@ -156,10 +265,6 @@ class AwsCompileCloudFrontEvents {
       if (functionObj.events) {
         functionObj.events.forEach(event => {
           if (event.cloudFront) {
-            if (!_.isObject(event.cloudFront)) {
-              throw new Error('cloudFront event has to be an object');
-            }
-
             const lambdaFunctionLogicalId = Object.keys(Resources).find(
               key =>
                 Resources[key].Type === 'AWS::Lambda::Function' &&

lib/plugins/aws/package/compile/events/cloudFront/index.test.js

@@ -351,32 +351,6 @@ describe('AwsCompileCloudFrontEvents', () => {
   });
 
   describe('#compileCloudFrontEvents()', () => {
-    it('should throw an error if cloudFront event type is not an object', () => {
-      awsCompileCloudFrontEvents.serverless.service.functions = {
-        first: {
-          events: [
-            {
-              cloudFront: 42,
-            },
-          ],
-        },
-      };
-
-      expect(() => awsCompileCloudFrontEvents.compileCloudFrontEvents()).to.throw(Error);
-
-      awsCompileCloudFrontEvents.serverless.service.functions = {
-        first: {
-          events: [
-            {
-              cloudFront: 'some',
-            },
-          ],
-        },
-      };
-
-      expect(() => awsCompileCloudFrontEvents.compileCloudFrontEvents()).to.throw(Error);
-    });
-
     it('should throw an error if the region is not us-east-1', () => {
       options.region = 'eu-central-1';
       awsCompileCloudFrontEvents.serverless.service.functions = {