fix: create spec compliant default OpenAPI.servers fields if omitted (#…

…3220) This change is specific to OpenAPI 3.x.y definitions. Refs #3143
swagger-api · Nov 2, 2023 · becdf3c · becdf3c
1 parent fff622f
commit becdf3c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/execute/index.js b/src/execute/index.js
@@ -343,7 +343,7 @@ function oas3BaseUrl({ spec, pathName, method, server, contextUrl, serverVariabl
     [selectedServerObj] = servers;
   }
 
-  if (selectedServerUrl.indexOf('{') > -1) {
+  if (selectedServerUrl.includes('{')) {
     // do variable substitution
     const varNames = getVariableTemplateNames(selectedServerUrl);
     varNames.forEach((vari) => {

diff --git a/src/index.js b/src/index.js
@@ -144,9 +144,9 @@ Swagger.prototype.applyDefaults = function applyDefaults() {
     if (!spec.basePath) {
       spec.basePath = '/';
     }
-  } else if (isOpenAPI3(spec) && isHttpUrl(specUrl)) {
-    if (!spec.servers) {
-      spec.servers = [{ url: specUrl }];
+  } else if (isOpenAPI3(spec)) {
+    if (!spec.servers || (Array.isArray(spec.servers) && spec.servers.length === 0)) {
+      spec.servers = [{ url: '/' }];
     }
   }
 };