Improve http client aspect uri mask #21
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sometimes, the Span could be called inside a coroutine, then it will not have a tracer.root on its Context.
* add vscode and jetbrains ide folders into gitignore * refactor and add new patters and replacements * add tests
* Fix: TypeError & warnings * Fix: TypeError & warnings
fix: Defer function
fix: changing trace span name fix: changing trace span name
* fix: regex to catch OID longer than 24 caracteres * fix: license plate regex - License plate is considered valid if it has 7 or 8 numbers
* Otel Codec * adding annotation * removing strict_types to maintain compatibility * renaming parentId to spanId
* feat: add custom append methods for override * chore: change get root span method visibility * feat: allow append custom span
* [3.1]Bumps dependencies (#5796) Co-authored-by: 李铭昕 <715557344@qq.com> * Upgrade hyperf packages to version `~3.1.0` (#5801) * Bumps `phpunit` to `10.x` (#5802) Co-authored-by: 李铭昕 <715557344@qq.com> * Support swow psr7-plus interface for all components. (#5839) * Support aspect to `GuzzleHttp\Client::request()` (#5996) * Fixed the bug that `no_aspect` is overridden (#6013) * Allow using the tracer instance from context, append `Trace-Id` to Response Header (#6023) * Optimized the tracing in coroutine (#6027) Co-authored-by: 李铭昕 <715557344@qq.com> * Make `Hyperf\Coroutine\Traits\Container` as deprecated. (#6044) Co-authored-by: 李铭昕 <715557344@qq.com> * Using the tracer instance from coroutine context (#6046) * Supplement the missing tag configuration items (#6060) * Supplement the missing tag configuration items * Update CHANGELOG-3.0.md * Added `Kafka` reporter for zipkin (#6069) * Optimize kafka reporter (#6075) * Fix error that using non-zipkin driver of tracer (#6097) Co-authored-by: Deeka Wong <8337659+huangdijia@users.noreply.github.com> * Optimize kafka reporter of tracer (#6098) * Optimize the HttpClientFactory of tracer (#6100) * Allowed output log when an exception occurs (#6111) * Adds `RequestTraceListener` for `hyperf/tracer` (#6062) * Added `ignore_exceptions` for tracer (#6143) Co-authored-by: Deeka Wong <huangdijia@gmail.com> * Record the exception message only by opening the exception switch and closing the ignore switch when using `trace`. (#6157) Co-authored-by: 李铭昕 <715557344@qq.com> * Upgrade actions/checkout to v4 (#6159) * Optimize `KafkaClientFactory` of `hyperf/tracer` (#6167) * Added RpcAspect and use it instead of JsonRpcAspect (#6198) * Added switch for `ElasticserachAspect` and `CoroutineAspect`. (#6200) * Added `Hyperf\Tracer\Aspect\GrpcAspect` (#6203) Co-authored-by: Deeka Wong <8337659+huangdijia@users.noreply.github.com> * Optimize the tracer component by replacing the direct instantiation with a closure at Context::getOrSet. (#6315) Co-authored-by: 李铭昕 <715557344@qq.com> * Changed the `branch-alias` of all components to `3.1-dev`. (#6320) * Fixed bug that the jaeger cannot show the http code when using `tracer`. (#6321) Co-authored-by: 李铭昕 <715557344@qq.com> * Use `Hyperf\Coroutine\Coroutine` instead of `Hyperf\Engine\Coroutine` (#6513) * Release v3.1.8 (#6515) Co-authored-by: Sharif <54396379+developersharif@users.noreply.github.com> Co-authored-by: guandeng <guandeng@Hotmail.com> Co-authored-by: Luffy <52o@qq52o.cn> * Added default config of noop driver for `hyperf/opentracing`. (#6550) * fix: noop tracer test (#6552) * Fixed implicitly nullable params (#6616) * Format code by the latest `cs-fixer`. (#6617) * chore: Update phpstan.neon.dist to level 6 (#6780) Co-authored-by: 李铭昕 <715557344@qq.com> * Added the request body and response body to the tracer (#6793) * Added composer.json normalize support (#6887) * Improve actions (#7088) * update github actions for all components. (#7090) * remove hyperf/utils compoent (#7411) * Fixed error versions for `hyperf/engine`. (#7429) * merge from main lib; resolving conflicts * bring recursive get root to TracerContext; resolving conflicts. * revert traceId with uuid creation; fix Typo; Removing workflow not needed; fix docblocks. * Fix Elasticsearch Typo (#7468) * fix import * fix http client aspect; fix import. * Update src/Aspect/HttpClientAspect.php Co-authored-by: Leonardo Teixeira <leooli.teixeira@gmail.com> --------- Co-authored-by: Deeka Wong <huangdijia@gmail.com> Co-authored-by: 李铭昕 <715557344@qq.com> Co-authored-by: 张城铭 <z@hyperf.io> Co-authored-by: crayxn <crayxn@qq.com> Co-authored-by: Deeka Wong <8337659+huangdijia@users.noreply.github.com> Co-authored-by: 宣言就是Siam <59419979@qq.com> Co-authored-by: lixinhan <327902941@qq.com> Co-authored-by: CodeWay <xiaoliang3275@163.com> Co-authored-by: Sharif <54396379+developersharif@users.noreply.github.com> Co-authored-by: guandeng <guandeng@Hotmail.com> Co-authored-by: Luffy <52o@qq52o.cn> Co-authored-by: Weslen Teche <weslen_teche@hotmail.com> Co-authored-by: jonas-elias <48037643+jonas-elias@users.noreply.github.com> Co-authored-by: Leonardo Teixeira <leooli.teixeira@gmail.com>
|
Hi, this is a READ-ONLY repository, please submit your PR on the https://github.com/hyperf/hyperf repository. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.