getsentry / getsentry/sentry-javascript

node-integration-tests: port HTTP and outgoing request suites to span streaming

オープン
#24,135 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
javascript Task
主要言語
TypeScript
スター
8.7k
フォーク
1.8k
平均マージ
1日 17時間
マージ済み PR(30日)
523

説明

Follow the porting recipe in the parent issue.

Highest churn group, because `http.server` and `http.client` span names became low cardinality. Do this one first, since the other groups inherit its patterns.

## Suites

* `tracing/requests/*` (15 dirs)
* `tracing/http-client-spans/*` (8 dirs)
* `tracing/httpIntegration`
* `tracing/tracePropagationTargets` and `tracing/tracePropagationTargets/baggage-org-id`
* `proxy`
* `ipv6`

## Twins to fold back

* `tracing/httpIntegration-streamed` into `tracing/httpIntegration`
* `tracing/http-client-spans/fetch-basic-streamed` into `tracing/http-client-spans/fetch-basic`

`tracing/http-client-span-streamed`, `tracing/httpServerSpans-streamed-unrouted` and `tracing/requestData-streamed` have no static original. Rename them to drop the `-streamed` suffix once nothing collides.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start with the parent issue's porting recipe, then inspect the listed tracing, proxy, and ipv6 suites and their streamed twins. Follow the recipe across the named directories, fold back the two specified twins, and rename the three streamed-only suites when there are no collisions. Done means the listed HTTP and outgoing-request suites are ported consistently without duplicate suite names.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
node.js, typescript
領域
api, backend, testing
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
活発
明瞭さ
おおむね明確
初心者へのやさしさ
48/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。