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

tabFilter no longer filters tabs

Abierto
#385 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
TypeScript
Estrellas
447
Forks
76
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza localizando la condición de filtrado de pestañas introducida en el commit a29df5f y siguiendo cómo se aplica config.tabFilter a los descriptores de procesos y pestañas. Reproduce el problema con filtros de inclusión y exclusión, y verifica después que las pestañas solo se adjunten cuando sus URLs cumplan los filtros configurados.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
typescript
Área
devtools, tooling
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.