<@UFT11FKSP> I've created PR (<https://github.com/rjaros/kvision/pull/266>). It's a draft and I will...

# kvision

[Tomas Kormanak]

@Robert Jaros I've created PR (https://github.com/rjaros/kvision/pull/266). It's a draft and I will add more states if you like it.

[Tomas Kormanak]

We often need to catch 404 exception. It is a valid case that object does not exist. Current implementation does not distinguish between error states and does not expose status code.

[Robert Jaros]

Nice work! 🙂

[Tomas Kormanak]

Thanks, I updated MR, ready to merge.

[Robert Jaros]

Merged. Thank you!

[Tomas Kormanak]

I created another one with some improvments.

[Robert Jaros]

The exception is still ServiceUnaviable

[Tomas Kormanak]

I've fixed it.

[Robert Jaros]

Hi. We have unintentionally introduced breaking change with this change 😞 I just have error reports from application in production 🙂

[Tomas Kormanak]

How?

[Robert Jaros]

I haven't noticed your RemoteRequestException extends Throwable but not Exception

[Robert Jaros]

so my typical code is failing:

try {
 restClient......
} catch (e: Exception) {
 ....
}

[Robert Jaros]

Could you make new PR and change Throwable to Exception?

[Tomas Kormanak]

Oh, sure.

[Robert Jaros]

Thx!

[Robert Jaros]

I'll make a bugfix release today.

[Tomas Kormanak]

Anyway it might be better to catch specific exception than just Exception