happy flow statuscodes 3xx, 4xx, 5xx considered an error?

Tapir version: 1.2.12

Scala version: 2.13.10

A 3xx, 4xx and 5xx status codes are always considered an error when put into the happy flow (.out).

I think all happy flow definitions should be parsed as right-flow, not left. Consider:

..: PublicEndpoint[Unit, Unit, String, Any]
---
.out(statusCode(StatusCode.TemporaryRedirect))
.out(
  header[String]("Location").description("Redirect address")
)

I would expect this endpoint definition to do a call and get back a 307 response with a header. I expect the 307 response and everything else I want to consider an error. The openapi spec will correctly list the response as one-of the possible response (including all listed 4xx and 5xx response).
So also:

..: PublicEndpoint[Unit, Unit, String, Any]
---
.out(statusCode(StatusCode.NotFound))
.out(
  header[String]("MyCustomHeader").description("Some value")
)

should follow a right-flow when used with tapir clients.

Only when the response does not fit within the right-flow it should go left and attempt decoding using the defined errorOut types.

What do you think?

Start from the two endpoint examples in the issue and trace how Tapir clients classify responses returned by .out for 3xx, 4xx, and 5xx status codes. Compare that behavior with responses decoded through errorOut and verify that the intended right-flow responses, headers, and OpenAPI response listings remain correct.