filesystem: a dangling symlink is reported as "Parent directory does not exist"
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
El fallo se describe en src/filesystem/lib.ts, y src/filesystem/__tests__/server-paths.test.ts fija el mensaje actual. Lee primero validatePath y las pruebas existentes de validación de rutas. Actualiza el error para un enlace simbólico cuyo destino no existe y, después, confirma que la prueba espera un mensaje sobre un enlace simbólico roto mientras la solicitud sigue siendo rechazada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
When the requested path is a symlink whose target does not exist, fs.realpath in validatePath (src/filesystem/lib.ts) fails with ENOENT and the code falls through to resolveUnicodeEquivalentPath, whose own realpath of the entry fails as well. validatePath reports that as "Parent directory does not exist:
src/filesystem/__tests__/server-paths.test.ts pins the message.
Expected
The error says the path is a broken symlink. Refusing the request is correct and stays.
Prior work
- #4915 by @v0ropaev
The PR(s) above targeted the pre-refactor code and are closed as part of the backlog triage in #4875; this issue is where the work continues, with the PR(s) as a prototype. Thank you to their authors.
- Lenguaje dominante
- TypeScript
- Estrellas
- 91.6k
- Forks
- 11.9k
- Merge medio
- 6 h 37 min
- PR fusionados (30 d)
- 147
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 modelcontextprotocol/servers
-
Clear the four CodeQL alerts in the release scripts (replaceAll for %2F, comment-line check)Abiertochore v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
modelcontextprotocol/servers#5261 ·
Los mantenedores suelen responder en 1 día
-
docs(git): clarify unconfined default scope and repository confinement options in README and CLIAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
modelcontextprotocol/servers#5231 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
docs(git): clarify unconfined default scope and repository confinement options in README and CLIAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100
modelcontextprotocol/servers#5230 ·
Los mantenedores suelen responder en 1 día
-
git: README should state plainly that without --repository every reachable repository is in scopeAbiertodocumentation server-git v2
Dificultad 1/5 1-3 horas Aptitud para principiantes 84/100
modelcontextprotocol/servers#5225 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
chore v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
modelcontextprotocol/servers#5223 ·
Los mantenedores suelen responder en 1 día
Todos los issues de modelcontextprotocol/servers
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
awslabs/visual-asset-management-system#804 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
restatedev/sdk-typescript#813 ·
-
bug desktop P1 session XS
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Merit-Systems/x402scan#1312 ·
-
type:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Azure/apiops-cli#331 ·
Los mantenedores suelen responder en 3 días