Skip to content

Feat Add TLS & mTLS support for gRPC with root CA and insecure mode (…#1363

Merged
cicoyle merged 1 commit into
dapr:release-1.15from
cicoyle:cherry-pick-grpc-ssl
May 14, 2025
Merged

Feat Add TLS & mTLS support for gRPC with root CA and insecure mode (…#1363
cicoyle merged 1 commit into
dapr:release-1.15from
cicoyle:cherry-pick-grpc-ssl

Conversation

@cicoyle

@cicoyle cicoyle commented May 14, 2025

Copy link
Copy Markdown
Contributor

cherry pick #1361

…apr#1361)

* feat: Support for GRPC ssl

Signed-off-by: Javier Aliaga <[email protected]>

* add tests

Signed-off-by: Cassandra Coyle <[email protected]>

* fix CI

Signed-off-by: Cassandra Coyle <[email protected]>

* add back else if

Signed-off-by: Cassandra Coyle <[email protected]>

* channel cleanup

Signed-off-by: Cassandra Coyle <[email protected]>

* add root ca support

Signed-off-by: Cassandra Coyle <[email protected]>

* checkstyles

Signed-off-by: Cassandra Coyle <[email protected]>

* add insecure

Signed-off-by: Cassandra Coyle <[email protected]>

* fix checkstyles

Signed-off-by: Cassandra Coyle <[email protected]>

* use InsecureTrustManagerFactory

Signed-off-by: Cassandra Coyle <[email protected]>

* fix test

Signed-off-by: Cassandra Coyle <[email protected]>

---------

Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: Cassandra Coyle <[email protected]>
Co-authored-by: Javier Aliaga <[email protected]>
@cicoyle
cicoyle requested review from a team as code owners May 14, 2025 22:14
@cicoyle
cicoyle merged commit c9c13b3 into dapr:release-1.15 May 14, 2025
@codecov

codecov Bot commented May 14, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 87.69231% with 8 lines in your changes missing coverage. Please review.

Please upload report for BASE (release-1.15@0b38982). Learn more about missing BASE report.

Files with missing lines Patch % Lines
sdk/src/main/java/io/dapr/utils/NetworkUtils.java 86.66% 5 Missing and 3 partials ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##             release-1.15    #1363   +/-   ##
===============================================
  Coverage                ?   77.46%           
  Complexity              ?     1773           
===============================================
  Files                   ?      204           
  Lines                   ?     5445           
  Branches                ?      597           
===============================================
  Hits                    ?     4218           
  Misses                  ?      908           
  Partials                ?      319           

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

siri-varma pushed a commit to siri-varma/java-sdk that referenced this pull request Aug 12, 2025
…apr#1361) (dapr#1363)

* feat: Support for GRPC ssl

* add tests

* fix CI

* add back else if

* channel cleanup

* add root ca support

* checkstyles

* add insecure

* fix checkstyles

* use InsecureTrustManagerFactory

* fix test

---------

Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: Cassandra Coyle <[email protected]>
Co-authored-by: Javier Aliaga <[email protected]>
Signed-off-by: siri-varma <[email protected]>
siri-varma pushed a commit to siri-varma/java-sdk that referenced this pull request Aug 12, 2025
…apr#1361) (dapr#1363)

* feat: Support for GRPC ssl

* add tests

* fix CI

* add back else if

* channel cleanup

* add root ca support

* checkstyles

* add insecure

* fix checkstyles

* use InsecureTrustManagerFactory

* fix test

---------

Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: Cassandra Coyle <[email protected]>
Co-authored-by: Javier Aliaga <[email protected]>
Signed-off-by: siri-varma <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants