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

3.0 #14

Closed
wants to merge 3 commits into from
Closed

3.0 #14

wants to merge 3 commits into from

Conversation

almostless
Copy link

处理了链路追踪使用zipkin时的3个问题:
1.requesterFactory之前试图替换为hyperf的httpClientFactory。但是没有生效。

2.zipkin上报后,服务端的响应可以是200,也可以是202,但是目前限制死了只能为202,导致每次服务端正确响应后,都new 了一个Runtime异常。我查看了openzipkin/zipkin-go客户端,它允许服务端响应状态码为200~299。 从实际出发,阿里云、火山云的链路追踪,上报后的http状态码是200而不是202。

3.创建http上报对象时,传入了logger类,便于发生了Runtime异常时定位原因

@github-actions
Copy link

Hi, this is a READ-ONLY repository, please submit your PR on the https://github.com/hyperf/hyperf repository.

This Pull Request will close automatically.

Thanks!

@github-actions github-actions bot closed this Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant