Skip to content
GitLab
Closed
Issue created by Administrator@rootContributor3 of 6 checklist items completed3/6 checklist items

[GO][BUG] *_api.go CRUD function return error instead of GenericOpenAPIError

Created by: code-lucidal58

Bug Report Checklist

  • Have you provided a full/minimal spec to reproduce the issue?
  • Have you validated the input using an OpenAPI validator (example)?
  • Have you tested with the latest master to confirm the issue still exists?
  • Have you searched for related issues/PRs?
  • What's the actual output vs expected output?
  • [Optional] Sponsorship to speed up the bug fix or feature request (example)
Description

In go client, *_api.go files, Execute() return 3 parameters, the last one being error. Since this of type error, response body is not received. Instead, only the Status code is sent. Plus once the response is read in the Execute(), it cannot be read again. (Well, this is how it works) Sending the second parameter nethttp.Response is of not much use because it has the body missing.

In all, the response from server in case of a failure is never sent back to the user.

openapi-generator version

v5.0.0-beta3

OpenAPI declaration file content or url
Suggest a fix

Use of GenericOpenAPIError as return type of 3rd parameter in the Execute function. Use NopCloser from ioutil to repopulate the body of response, so that it can be used later.