-
Notifications
You must be signed in to change notification settings - Fork 6
Propagate error instance ID from remote errors #235
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
Conversation
Generate changelog in
|
312dcd9 to
8b014a3
Compare
8b014a3 to
082d2ab
Compare
|
This PR has been automatically marked as stale because it has not been touched in the last 14 days. If you'd like to keep it open, please leave a comment or add the 'long-lived' label, otherwise it'll be closed in 7 days. |
|
The error instance ID of remote errors is now propagated. (cherry picked from commit a3441a3)
Before this PR
By default, Conjure errors received from other services would turn into an error that would not propagate any of the upstream info, including its instance ID. This doesn't match the behavior of conjure-java as of palantir/conjure-java-runtime-api#348.
After this PR
==COMMIT_MSG==
The error instance ID of remote errors is now propagated.
==COMMIT_MSG==