Allow passing baseURL to Swagger.exeute method.

[sauravazad]

Content & configuration

Swagger-Client usage:

SwaggerClient.execute({
 baseURL: 'someURL'
})

Is your feature request related to a problem?

baseURL is computed every time when an execute is invoked .

swagger-js/src/execute/index.js

Line 313 in ee7fdd6

export function baseUrl(obj) {

Please have a look at below flame chart .
Building the URL consumes almost 75% of execution time for the executeClient method .

Describe the solution you'd like

If SwaggerClient.execute() were to accept a baseURL , and use it if provided instead of building it , It would help in improving the performance and execution time .

Please let me know if this added support is acceptable . If so, I will raise a PR to add the support.

[sauravazad]

@char0n

Any comments or thought on the above proposal ?

[char0n]

Hi @sauravazad,

The proposal is reasonable. We're gonna go for it. I'll tend the the PR ASAP.

Thank you!