firefox-devtools / firefox-devtools/vscode-firefox-debug

tabFilter no longer filters tabs

オープン
#385 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
TypeScript
スター
447
フォーク
76
PR マージ指標
30日以内にマージされた PR はありません

説明

Setting the `tabFilter` property in the config or relying on the automatically generated `tabFilter` per the documentation does not actually result in any filtered tabs. All tabs are attached regardless of filters. This gums up my console log with all console messages from all tabs I have open, rather than just the ones relevant to the app I'm trying to debug, which is quite annoying.

This seems to be a regression introduced in a29df5f. The `tabFilter` config option is only verified against processes, not tabs:


```js
const url = targetActor.target.url;
if (
descriptorActor.type === 'process' && !targetActor.target.addonId && url &&
(!this.config.tabFilter.include.some(tabFilter => tabFilter.test(url)) ||
this.config.tabFilter.exclude.some(tabFilter => tabFilter.test(url)))
) {
skip = true;
}
```

This is allowing all tabs through regardless of URL.

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

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

調査の方向性

まず、コミット a29df5f で導入されたタブフィルタリング条件を見つけ、config.tabFilter がプロセスおよびタブのディスクリプタにどのように適用されているかを追跡します。include フィルタと exclude フィルタを使って問題を再現し、その後、URL が設定されたフィルタを満たす場合にのみタブが関連付けられることを確認します。

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

評価

技術スタック
typescript
領域
devtools, tooling
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

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

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