adobe / adobe/aio-lib-java

`core` improve `FeignUtil` `BaseBuilder` to allow the client to choose the `Feign` logger as well as the `Feign` `LogLevel`

Open
#146 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
7
Forks
22
PR merge metrics
No merged PRs in 30d

Description

improve `FeignUtil` `BaseBuilder` to allow the client to choose
* its `Feign` logger
* as well as the `Feign` `LogLevel`

see https://github.com/adobe/aio-lib-java/pull/145#discussion_r1094850264

It looks like the Feign Logger API is under refactoring see
* https://github.com/OpenFeign/feign#short-term---what-were-working-on-now-
* https://github.com/OpenFeign/feign/issues?q=is%3Aissue+is%3Aopen+logger+label%3Afeign-12
* https://github.com/OpenFeign/feign/issues/1143

a few Feign logging related articles:
* https://cloud.spring.io/spring-cloud-openfeign/reference/html/#feign-logging
* https://medium.com/@piszu/feign-client-is-not-so-simple-as-we-want-case-study-920e4901e27b

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.