Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core: Feign Implementation Details exposed to clients #164

Open
bstopp opened this issue Aug 4, 2023 · 0 comments
Open

core: Feign Implementation Details exposed to clients #164

bstopp opened this issue Aug 4, 2023 · 0 comments

Comments

@bstopp
Copy link
Collaborator

bstopp commented Aug 4, 2023

The use of the Feign library/framework is an implementation detail and shouldn't concern the consumers of the library.

However, every API call throws runtime exceptions all of which extend Feign's. That they are Runtime does abstract away many concerns. However, the APIs should throw declared exceptions (Runtime or not) so that consumers can properly handle error states, without relying on inspecting the underlying error details.

@francoisledroff francoisledroff changed the title Feign Implementation Details exposed to clients core: Feign Implementation Details exposed to clients Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant