Fix the warn log issue caused by not setting the dapr.client.grpcPort…

[seal90]

… variable

Description

Please explain the changes you've made
Add null verification for io.dapr.spring.boot.autoconfigure.client.DaprClientAutoConfiguration#createPropertiesFromConnectionDetails

  private Properties createPropertiesFromConnectionDetails(DaprConnectionDetails daprConnectionDetails) {
    final Map<String, String> propertyOverrides = new HashMap<>();
    propertyOverrides.put(Properties.HTTP_ENDPOINT.getName(), daprConnectionDetails.httpEndpoint());
    if(daprConnectionDetails.httpPort() != null) {
      propertyOverrides.put(Properties.HTTP_PORT.getName(), String.valueOf(daprConnectionDetails.httpPort()));
    }
    propertyOverrides.put(Properties.GRPC_ENDPOINT.getName(), daprConnectionDetails.grpcEndpoint());
    if(daprConnectionDetails.grpcPort() != null) {
      propertyOverrides.put(Properties.GRPC_PORT.getName(), String.valueOf(daprConnectionDetails.grpcPort()));
    }
    return new Properties(propertyOverrides);
  }

When using the grpc protocol without configuring dapr.client.grpcPort, the following warn information will be printed.This is because put("dapr.grpc.port", "null") in the above code causes DaprClientAutoConfiguration#daprWorkflowClient to use Property#get to parse the configuration and fail to handle the string "null"

2025-03-29T09:49:14.430+08:00  WARN 4660 --- [dapr-demo] [           main] io.dapr.config.Property                  : Invalid override value in property: dapr.grpc.port
2025-03-29T09:49:15.080+08:00  WARN 4660 --- [dapr-demo] [           main] io.dapr.config.Property                  : Invalid override value in property: dapr.grpc.port
2025-03-29T09:49:15.594+08:00  WARN 4660 --- [dapr-demo] [           main] io.dapr.config.Property                  : Invalid override value in property: dapr.grpc.port
2025-03-29T09:49:16.474+08:00  WARN 4660 --- [dapr-demo] [           main] io.dapr.config.Property                  : Invalid override value in property: dapr.grpc.port

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: #1292

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]

@seal90 thanks a lot for your contribution!

I have a few asks:

• Could you please open an issue for your fix. I know it might sound as bureaucratic, but we want to make sure all the fixes have proper paper trail
• Could you please make sure that you sign your commits so the DCO task doesn't fail.

Thank you.

[artur-ciocanu]

Lovely! Thanks for the fix!

[mcruzdev]

Nice catch!

[mcruzdev]

Errors are related to checkstyle.

[seal90]

Okay I got it.

• issue The warn log issue caused by not setting the dapr.client.grpcPort variable #1292
• fix gpg check
• fix checkstyle

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 75.47%. Comparing base (d759c53) to head (f35c63b).
Report is 123 commits behind head on master.

Files with missing lines	Patch %	Lines
...oconfigure/client/DaprClientAutoConfiguration.java	0.00%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1289      +/-   ##
============================================
- Coverage     76.91%   75.47%   -1.44%     
- Complexity     1592     1616      +24     
============================================
  Files           145      193      +48     
  Lines          4843     5101     +258     
  Branches        562      555       -7     
============================================
+ Hits           3725     3850     +125     
- Misses          821      935     +114     
- Partials        297      316      +19     

☔ 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.