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.

Sign up for GitHub

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

More logs #29

Merged
merged 11 commits into from
Sep 9, 2021
Merged

More logs #29

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
f01ece8
fix: add triple attachmen support
LaurenceLiZhixin Sep 4, 2021
014b628
fix: import fmt
LaurenceLiZhixin Sep 4, 2021
471bc94
fix: fmt
LaurenceLiZhixin Sep 4, 2021
ff84f83
fix: api
LaurenceLiZhixin Sep 4, 2021
250f98d
fix: atta ci
LaurenceLiZhixin Sep 4, 2021
4acfbbd
fix ci
LaurenceLiZhixin Sep 4, 2021
4bea0e5
fix: conflict
LaurenceLiZhixin Sep 4, 2021
2ca1187
fix
LaurenceLiZhixin Sep 4, 2021
5748f95
fix: add logs
LaurenceLiZhixin Sep 9, 2021
f528ed7
Merge remote-tracking branch 'origin/master' into attachmentci
LaurenceLiZhixin Sep 9, 2021
0d83f21
fix: import format
LaurenceLiZhixin Sep 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix
  • Loading branch information
@LaurenceLiZhixin
LaurenceLiZhixin committed Sep 4, 2021
commit 2ca11876c86c9d41146116865896baaa076a39c3
1 change: 0 additions & 1 deletion internal/http2/triple_controller.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -399,7 +399,6 @@ func (hc *TripleController) UnaryInvoke(ctx context.Context, path string, arg, r
return *common.NewErrorWithAttachment(err, attachment)
}


headerHandler, _ := common.GetProtocolHeaderHandler(hc.option, ctx)
newHeader := http.Header{}
newHeader = headerHandler.WriteTripleReqHeaderField(newHeader)
Expand Down