[Spring] Fixes #5244 Include basePath @RequestMapping in Spring API template

[kerkhofsd]

PR checklist

• Read the contribution guidelines.
• Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\.
• Filed the PR against the correct branch: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
• Copied the technical committee to review the pull request if your PR is targeting a particular programming language.
  @bbdouglas (2017/07) @JFCote (2017/08) @sreeshas (2017/08) @jfiala (2017/08) @lukoyanov (2017/09) @cbornet (2017/09) @jeff9finger (2018/01)

Description of the PR

Add @RequestMapping(value = "{{{contextPath}}}") on class level in the api.mustache template to correctly support the basePath swagger parameter.

To make this work for spring-cloud, I had to introduce a mustache variable hostWithoutBasePath and use this in the spring-cloud apiClient.mustache template.

[jawwadfarooq]

When will a new version i.e. v2.2.4 release with this fix?

[IsaacDeLaRosa]

@kerkhofsd is this going to be merged anytime soon? what is the workaround? maybe something like a openapi.openAPIPetstore.base-path:/v2 var in the spring properties file similar to https://github.com/OpenAPITools/openapi-generator/blob/6a00b2a802c688a235539c8425924ad6287b49d6/samples/server/petstore/springboot-reactive/src/main/java/org/openapitools/api/AnotherFakeApiController.java#L9

[mikela10000]

@kerkhofsd i would really appreciate the merge of this.
My current workaround it to put manually the path in the RestControler, but this is a temporary fix.

@RestController
@RequestMapping(path = "/api/XXX/v1")
public class MyApplicationController implements SomeApi {
}

Is this PR for swagger 3.0 version ? currently still using 2.0.

[highlysignificantbit]

Should be nice if this were fixed! Doing the same now as @mikela10000 ...

[talaviss]

When will this PR will be merged?

[shark300]

Ping

[kerkhofsd]

As the creator of this PR (I'm just an open source contributor), I do not have any idea if this ever will be merged.
As everyone can see, I didn't receive any feedback and there was no effort to get this merged so far.

[sergeypavlikhin]

Is it possible to close and reopen this PR? This step will bump one

[HugoMario]

hi @kerkhofsd , sorry for this long and no fair delay. i'm going to check your changes and merge them. Thanks !!

[frantuma]

please build