Allow load balancer uri in uri attribute

Open antechrestos opened this issue 3 years ago • 4 comments

Description

Fixes #489 : This change is the one requested by late issue . The aim is to handle url such as lb://some-service (or `lbs://some-service). When such url is specified, we ensure load balance client is provided and we keep it instead of using delegate one).

Code implementation

I willingly kept 3 commit to better describe the implementation

improve code specification of name, url and path handle by adding tests
refactoring of code, particularly by forcing reentrance of code (url was modified by side effect)
implementing the feature

Mar 30 '21 17:03 antechrestos

Codecov Report

Merging #518 (5661a35) into master (bcea218) will increase coverage by 0.55%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #518      +/-   ##
============================================
+ Coverage     77.85%   78.41%   +0.55%     
- Complexity      489      499      +10     
============================================
  Files            60       60              
  Lines          1820     1816       -4     
  Branches        267      266       -1     
============================================
+ Hits           1417     1424       +7     
+ Misses          256      249       -7     
+ Partials        147      143       -4