This repository was archived by the owner on Feb 26, 2023. It is now read-only.
Repository navigation
This repository was archived by the owner on Feb 26, 2023. It is now read-only.
Add checked Exceptions on RestClient #508
Description
Activity
Just to be clear, this would mean:
- adding compile errors to the annotated
@Restinterfaces if they do not declare throwing the right checked exception - and therefore breaking backward compatibility on all
@Restinterfaces - Having to recreate exceptions corresponding to the already existing unchecked ones in the implementation frameworks
- Having to catch these runtime exceptions in the generated code and throw wrapping checked exceptions instead, leading to and additional layer in the stacktrace.
Maybe there's a good reason why Spring RestTemplate made these exceptions unchecked :)
- adding compile errors to the annotated
I admit that I didn't look into this thoroughly and it seems to complex for the usefulness of this feature.
Instead of all these code modifications and the presence of checked exception, it could just be documented then. WDYT?
Will do!
Any news about it @BluePyth ? :)
will do tomorrow ;)
Done, you can have a look at the "Handling error" subsection.
Seems correct to me. I'm closing this issue then :)
Metadata
Metadata
Assignees
Labels
No labels
Follow up for the discussion on the mailing list
The idea is that it's annoying to remember that RestClient throws Runtime Exceptions when there's either a JSON parsing problem or a problem with the server connection/response.