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

feat(volo-thrift): close connection when encounter error #482

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

PureWhiteWu
Copy link
Member

Mark crrst in ttheader when encounter error to tell client side to close connection.

@PureWhiteWu PureWhiteWu added A-volo-thrift The issue concerns the `volo-thrift` crate. C-enhancement This is a PR that adds a new feature or fixes a bug. labels Aug 7, 2024
@PureWhiteWu PureWhiteWu self-assigned this Aug 7, 2024
@PureWhiteWu PureWhiteWu requested review from a team as code owners August 7, 2024 06:34
@PureWhiteWu PureWhiteWu force-pushed the feat/support_crrst_when_exception branch 2 times, most recently from 7e77528 to 6a718bd Compare August 7, 2024 07:47
Millione
Millione previously approved these changes Aug 7, 2024
Millione
Millione previously approved these changes Aug 7, 2024
@PureWhiteWu PureWhiteWu force-pushed the feat/support_crrst_when_exception branch from 01347b9 to 4ee2608 Compare August 7, 2024 08:35
@PureWhiteWu PureWhiteWu merged commit 0c15227 into main Aug 7, 2024
16 checks passed
@PureWhiteWu PureWhiteWu deleted the feat/support_crrst_when_exception branch August 7, 2024 08:53
@Millione Millione mentioned this pull request Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-volo-thrift The issue concerns the `volo-thrift` crate. C-enhancement This is a PR that adds a new feature or fixes a bug.
Development

Successfully merging this pull request may close these issues.

2 participants