Properly append template to a base path with a trailing slash

[afflerbach]

see for example swagger-api/swagger-js#116

[Maks3w]

I'm about to decline this as per the specification you can infer all paths must always to start with a leading slash

basePath

The base path on which the API is served, which is relative to the host. If it is not included, the API is served directly under the host. The value MUST start with a leading slash (/). The basePath does not support path templating.

Path object

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. Path templating is allowed.

https://swagger.io/specification/v2/

[afflerbach]

This PR is about trailing slashes. With the current master it is currently not possible to validate any path if the base path is set to /. For example /my/path will be resolved to //my/path and won't ever match.

[Maks3w]

I understand. What I see basePath is not required.

If it is not included, the API is served directly under the host

So I think the correct is not publish a basePath when is on the root and may yo fix this library for to treat the as optional that attribute

[Maks3w]

https://github.com/swagger-api/swagger-js/blob/a635e37c3d9b3c85fbbe28f9c4051d5d4c1ea65c/src/execute/index.js#L355

[afflerbach]

If last character is '/', trim it off
https://github.com/swagger-api/swagger-js/blob/a635e37c3d9b3c85fbbe28f9c4051d5d4c1ea65c/src/execute/index.js#L367