Skip to content

Conversation

@shenqidebaozi
Copy link
Member

Description (what this PR does / why we need it):

Which issue(s) this PR fixes (resolves / be part of):

Other special notes for the reviewers:

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 22, 2024
@dosubot dosubot bot added the LGTM label Oct 22, 2024
@shenqidebaozi shenqidebaozi merged commit 4e9dac0 into main Oct 23, 2024
63 checks passed
@shenqidebaozi shenqidebaozi deleted the fix/errors_code_default branch October 23, 2024 12:49
chaosue pushed a commit to chaosue/kratos that referenced this pull request Dec 18, 2024
…o-kratos#3447)

* fix: resolve the issue of metrics and logging code default to zero

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

Labels

LGTM size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Question] Middleware appears to use HTTP error codes even when the request was originally gRPC - is this intended?

4 participants