Fix URL building logic

[artur-ciocanu]

Description

This is a tiny PR that fixes the URL building logic in the DaprHttp functionality. This is trying to fix an issue where we tried to replace the OkHttp URL builder with something that uses JDK URI.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1314

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Extended the documentation

[artur-ciocanu]

@rahulpoddar-fyndna and @vikram-mahadevkar this is the fix for #1314. We hope to release it as soon as possible.

[artur-ciocanu]

@cicoyle @mcruzdev @salaboy could you please take a look. This might need a hot fix.

[siri-varma]

LGTM

[artur-ciocanu]

@cicoyle @salaboy and @mcruzdev I have figured what was the problem with my setup. Here is the breakdown:

• To be able to use named parameters in Spring we have to either adjust the Java compiler arguments or make sure that we use @RequestParam("uri") String uri, otherwise this results in runtime exceptions and eventually ends up as HTTP 500.
• On my local box mDNS is not working, hence service to service was failing. I have no idea how to fix it, the way I managed to overcome this is by using SQLite name resolution.

After I have went through all this hurdles, I was able to confirm that my fix works and there are no more double encodings.

@cicoyle and @salaboy could you please review and approve. Thank you!

CC: @rdosifyndna

[artur-ciocanu]

@cicoyle and @salaboy since the name resolution based on mDNS can be such a pain should we use SQLite name resolution in our ITs. I think this should make our tests a little bit more predictable and reliable.

Any thoughts?

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 5 lines in your changes missing coverage. Please review.

Project coverage is 77.11%. Comparing base (d759c53) to head (74a769e).
Report is 134 commits behind head on master.

Files with missing lines	Patch %	Lines
sdk/src/main/java/io/dapr/client/DaprHttp.java	50.00%	3 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1320      +/-   ##
============================================
+ Coverage     76.91%   77.11%   +0.19%     
- Complexity     1592     1729     +137     
============================================
  Files           145      200      +55     
  Lines          4843     5291     +448     
  Branches        562      579      +17     
============================================
+ Hits           3725     4080     +355     
- Misses          821      899      +78     
- Partials        297      312      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.