search

swagger-api / swagger-js

Javascript library to connect to swagger-enabled APIs via browser or nodejs
http://swagger.io
Apache License 2.0
2.62k stars 758 forks source link

refactor(http): refactor http client #3482

Closed glowcloud closed 5 months ago

glowcloud commented 5 months ago

Refs #3481

  1. Moved request and response serializing/encoding into separate files
  2. Moved the logic introduced by https://github.com/swagger-api/swagger-js/pull/3476 and https://github.com/swagger-api/swagger-js/pull/3474 to request builder
glowcloud commented 5 months ago
  1. For placement of response and request serializers - mergeInQueryOrForm is also used in one place in execute, but it's also for building requests, so it still seemed related to http
  2. I also thought about leaving index in http just for exports - let me know if that would be better and the functions should be moved to an additonal file, ex. http.js
  3. file and format functions are only used for request serializing/encoding and in tests, so also looks like it's an appropriate place for them to be
swagger-bot commented 5 months ago

:tada: This PR is included in version 3.27.3 :tada:

The release is available on:

Your semantic-release bot :package::rocket: