5.1.0 changes status exception behaviour

[bjoernmayer]

Hey there,

we recently tried to update from v5.0.0 to v5.1.0.

Unfortunately, the status exception handling changed behaviour in this release.

At some point in the code we throw an IllegalArgumentException. Then catch it, and map it to a StatusException:

fun IllegalArgumentException.toGrpcFailedPreconditionException(): StatusException =
    Status.FAILED_PRECONDITION
        .withCause(this)
        .withDescription("Milk is expired")
        .asException()

This exception is then thrown in our grpc service.

We would expect this response:

ERROR:
  Code: FailedPrecondition
  Message: Milk is expired

The actual output is:

ERROR:
  Code: Internal
  Message: 

Wild guess: This one might cause this: #344

[jvmlet]

Which log messages do you see from this class ?

[bjoernmayer]

Closing call with Status{code=INTERNAL, description=null, cause=null}

Important to notice: The exception being thrown is not a RuntimeException.

[jvmlet]

Closing call with Status{code=INTERNAL, description=null, cause=null}

This is THE ONLY log you see ?

[bjoernmayer]

Yep. I even activated debug log level for org.lognet.springboot.grpc

[jvmlet]

Please try with latest 5.1.5-SNAPSHOT

[jvmlet]

5.1.5 was released