Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding SystemInfoInterceptor #320

Merged
merged 5 commits into from
Jul 11, 2023

Conversation

msmakouz
Copy link
Collaborator

@msmakouz msmakouz commented Jul 4, 2023

What was changed

Added Temporal\Client\Interceptor\SystemInfoInterceptor.

Why?

We need to know from the server what it supports. Specifically in this case, we need to stop retrying internal errors if the server says it differentiates between them.

Checklist

msmakouz added 2 commits June 28, 2023 15:16

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@roxblnfk roxblnfk requested review from Sushisource and roxblnfk July 4, 2023 16:33
msmakouz added 2 commits July 4, 2023 19:54

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@roxblnfk roxblnfk merged commit 3a17811 into temporalio:2.7.0 Jul 11, 2023
@rustatian rustatian added this to the 2.7.0 milestone Sep 1, 2023
@roxblnfk roxblnfk linked an issue Dec 13, 2023 that may be closed by this pull request
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.

Lazy connectivity for Temporal client and healthCheck
4 participants