firefox-devtools / firefox-devtools/vscode-firefox-debug
tabFilter no longer filters tabs
- 主要言語
- 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