Skip to content

chore: Deprecate invoke methods#1666

Open
javier-aliaga wants to merge 2 commits intodapr:masterfrom
javier-aliaga:deprecate-invoke-sdk-methods
Open

chore: Deprecate invoke methods#1666
javier-aliaga wants to merge 2 commits intodapr:masterfrom
javier-aliaga:deprecate-invoke-sdk-methods

Conversation

@javier-aliaga
Copy link
Contributor

Description

The guidance since 2023 has been to not use SDK invoke* methods for Service Invocation. However this has never been formalised by marking the methods as deprecated.

Instead it is recommended to use language native http clients and grpc clients for Service Invoke.

Factory methods in the SDK which return a native http/grpc client is fine too, and are not deprecated.

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

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

Signed-off-by: Javier Aliaga <javier@diagrid.io>
@javier-aliaga javier-aliaga requested review from a team as code owners February 19, 2026 08:48
@javier-aliaga
Copy link
Contributor Author

@olitomlinson can you review? Both method should be deprecated, isnt it? invokeMethod and invokeBinding?

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.27%. Comparing base (7ff08c5) to head (a4e9188).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1666   +/-   ##
=========================================
  Coverage     79.27%   79.27%           
  Complexity     2151     2151           
=========================================
  Files           234      234           
  Lines          6474     6474           
  Branches        703      703           
=========================================
  Hits           5132     5132           
  Misses          989      989           
  Partials        353      353           

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ALL SDKS] Deprecate Invoke* SDK methods used for Service Invocation

1 participant

Comments