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

tabFilter no longer filters tabs

Ouverte
#385 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
TypeScript
Étoiles
447
Forks
76
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par localiser la condition de filtrage des onglets introduite dans le commit a29df5f et suivez la manière dont config.tabFilter est appliqué aux descripteurs de processus et d’onglets. Reproduisez le problème avec des filtres d’inclusion et d’exclusion, puis vérifiez que les onglets ne sont associés que lorsque leurs URL respectent les filtres configurés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
devtools, tooling
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.