-
Notifications
You must be signed in to change notification settings - Fork 760
Closed
Labels
P2pull-request-welcomeThat the core team don't have the resources, but would welcome a community PRThat the core team don't have the resources, but would welcome a community PRtype: enhancement
Description
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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P2pull-request-welcomeThat the core team don't have the resources, but would welcome a community PRThat the core team don't have the resources, but would welcome a community PRtype: enhancement