📣 Folks, would appreciate your feedback on this! https://twitter.com/JetBrainsKtor/status/1466061672481796103 📣

Can you share insight into the current debate for why some feel it's worth changing?

I'd like to keep exceptions for client calls that are returning extracted and deserialized response body, but not to throw exceptions when returning some abstraction container over http response (with headers, body, response code and other metadata)

@Trevor Stone we observe a good number of users setting this value to False (i.e. changing the default). This is based on internal usage at JetBrains as well as the occasional code we see in samples, etc. And also a few of us on the team kind of feel like anything outside of 2XX is still a valid status code so shouldn’t be treated as an exception (which is really for exceptional cases).

I always set if to false, because not having 200 is not a runtime error it is an expected one if you use http status codes.

If only kotlin would have a distinction between the two. Almost like it needs "checked" exceptions :troll:

+1 for no exceptions, especially not if the remote server returned a response of any kind saves me from having to wrap those exceptions myself 😄

+1 for no exceptions, I always turn them off.

No exceptions, indeed I have adapters for both, retrofit and ktor returning an

Either<SomeError, SomeSuccess>

+1 for defaulting to not throwing