search

konrad-kaminski / spring-kotlin-coroutine

Kotlin coroutine support for Spring.
448 stars 69 forks source link

Unwrap InvocationTargetException on DeferredRestOperations #16

Closed Krotki closed 6 years ago

Krotki commented 6 years ago

I think DeferredRestOperations should return same exceptions as original RestOperations, so they can be used as drop in replacement.

Probably this change should be also applied for CoroutineRestOperations, but I wanted to know your opinion first.

konrad-kaminski commented 6 years ago

You're right. We should add the same behaviour to other MethodInvokerProvider implementations.

Krotki commented 6 years ago

@konrad-kaminski could you check whats wrong ? I tried to reproduce this locally, but it doesn't fail. Also I'm not sure how my changes could influence those tests.