exclude_file_regex and applies_to_file_regex only see the file name in devskim analyze
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
Línea de trabajo
Empieza en AnalyzeCommand.cs, alrededor de la línea 421, donde se pasa fileEntry.Name a DevSkimRuleProcessor.Analyze; sigue cómo construye un FileEntry y cómo ApplicationInspector compara las expresiones regulares de los archivos. Compara este comportamiento con la expectativa de ruta completa descrita en el issue #667. El trabajo estará terminado cuando la expresión regular de exclusión personalizada coincida con la ruta del archivo analizado, como se espera.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
With DevSkim 1.0.70, the file regexes of a custom rule are compared with the file name only, not with its path.
This rule never excludes project/tests/e2e/fixtures/prepare-ner.py:
"applies_to_file_regex": [".*"],
"exclude_file_regex": ["project/tests/"]
"exclude_file_regex": ["^prepare-ner\\.py$"] does exclude it.
AnalyzeCommand.cs (line 421) passes fileEntry.Name to DevSkimRuleProcessor.Analyze, which builds a new FileEntry from it. ApplicationInspector then matches the regexes against that name only. #667 says these regexes run against the full path, which is what I expected too. Could Analyze receive the path relative to the scanned folder?
- Lenguaje dominante
- C#
- Estrellas
- 1k
- Forks
- 133
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/DevSkim
-
The DevSkim Repository Description still mentions IDE extensions but these are not longer includedAbiertobug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 55/100
-
DS117838 and DS173237 report published sha256 digests as secretsPosiblemente ocupada @gfs la tomó hace 1 día. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 56/100
-
How to match files without a file extension?Posiblemente ocupada @gfs la tomó hace 167 días. Abiertoenhancement
microsoft/DevSkim#747 · 4 comentarios · 1 reacción · 2 asignados ·
-
Support for HTML (see attached example rule)Posiblemente ocupada @Guflly la tomó hace 88 días. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Todos los issues de microsoft/DevSkim
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
[Bug] Dashboard renders attachment links that resolve to 404 when the FILES binding is absentAbiertobug 優先: 低
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
.NET area: storage bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Larson-Logan/medal-sync#61 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día