Fixes for various JSON/non-JSON content issues

[alexrudall]

[BREAKING CHANGE]

• Try to parse all responses as JSON
• If parsing JSON fails, return the original response as-is
• Reproduction and fix for Fix - File content #544

To Test

Update gemfile:

gem 'ruby-openai', git: 'git://github.com/alexrudall/ruby-openai.git', branch: 'fix/non-json-content'

Run bundle install

Related Issues (?)

#277
#314
#416
#421
#562

Related PRs (?)

#474
#504
#540
#544

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Have you added an explanation of what your changes do and why you'd like us to include them?

[jprosevear]

Is this just about the files endpoint or is there others that don't return JSON?

[alexrudall]

@jprosevear Audio endpoints are another one

[jprosevear]

@jprosevear Audio endpoints are another one

My thought was to pass a named param to select the parsing option and default to json. Then change the default for files and audio to limit the breaking damage. Not sure if that's the right path though

[alexrudall]

@jprosevear Audio endpoints are another one

My thought was to pass a named param to select the parsing option and default to json. Then change the default for files and audio to limit the breaking damage. Not sure if that's the right path though

Yep, saw that in one of the PRs. I'd prefer just just always check if Strings can be JSON and always return raw if there are any errors - that way it should work for future endpoints, unforseen edge cases, etc. not sure of downsides to this approach

[alexrudall]

Released in ruby-openai v8