From 619734029db3cd00b44e18c7f5ca3dd5ddf0cd8b Mon Sep 17 00:00:00 2001 From: navinger Date: Thu, 29 Mar 2018 12:29:18 -0700 Subject: [PATCH] Remove basePath: "/" I'm updating all the openapi files in the getting-started sample in all the sample repos to remove basePath: "/" Here's the reason from simonz130: From the OpenAPI 2 spec: * basePath: "If it is not included, the API is served directly under the host. The value MUST start with a leading slash (/). " * Paths for methods: "A relative path to an individual endpoint. The field name MUST begin with a slash. The path is appended to the basePath in order to construct the full URL." This OpenAPI getting-started sample have basePath: "/", which (per strict spec interpretation) means all the paths start with double-slashes. (e.g "//v1/shelves" rather than "/v1/shelves"). Removing basepath="/" fixes that. --- endpoints/getting-started/openapi.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/endpoints/getting-started/openapi.yaml b/endpoints/getting-started/openapi.yaml index 840f2bd290..ed7a3a8865 100644 --- a/endpoints/getting-started/openapi.yaml +++ b/endpoints/getting-started/openapi.yaml @@ -6,7 +6,6 @@ info: version: "1.0.0" host: "echo-api.endpoints.YOUR-PROJECT-ID.cloud.goog" # [END swagger] -basePath: "/" consumes: - "application/json" produces: