OpenAPI's components object is optional

From discussion in this PR: #187 (comment)
cdimascio · Dec 18, 2019 · 9438b80 · 9438b80
1 parent c604025
commit 9438b80
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 6 deletions.
diff --git a/src/framework/ajv/index.ts b/src/framework/ajv/index.ts
@@ -85,7 +85,7 @@ function createAjv(
     });
   }
 
-  if (openApiSpec.components.schemas) {
+  if (openApiSpec.components?.schemas) {
     Object.entries(openApiSpec.components.schemas).forEach(
       ([id, schema]: any[]) => {
         ajv.addSchema(schema, `#/components/schemas/${id}`);

diff --git a/src/middlewares/openapi.request.validator.ts b/src/middlewares/openapi.request.validator.ts
@@ -99,7 +99,7 @@ export class RequestValidator {
 
     const securityQueryParameter = this.getSecurityQueryParams(
       usedSecuritySchema,
-      this._apiDocs.components.securitySchemes,
+      this._apiDocs.components?.securitySchemes,
     );
 
     let requestBody = pathSchema.requestBody;

diff --git a/src/middlewares/openapi.security.ts b/src/middlewares/openapi.security.ts
@@ -55,8 +55,7 @@ export function security(
       return next();
     }
 
-    const securitySchemes =
-      context.apiDoc.components && context.apiDoc.components.securitySchemes;
+    const securitySchemes = context.apiDoc.components?.securitySchemes;
 
     if (!securitySchemes) {
       const message = `security referenced at path ${path}, but not defined in 'components.securitySchemes'`;

diff --git a/test/resources/serialized.objects.yaml b/test/resources/serialized.objects.yaml
@@ -69,5 +69,3 @@ paths:
             application/json:
               schema:
                 type: object
-
-components: {}