`continuity.context.Walk` assumes incorrect behavior for `os.Readlink`
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- go
- Área
- operating-systems
Línea de trabajo
Comienza en el punto de entrada continuity.context.Walk e inspecciona el commit enlazado; después, compara su gestión de os.Readlink con el comportamiento de filepath.Walk y filepath.WalkDir. Usa las incidencias y cambios de Go referenciados para verificar el comportamiento POSIX esperado; se considera terminado cuando Walk funciona correctamente con un symlink raíz relativo y un symlink-a-symlink.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The code added in https://github.com/containerd/continuity/commit/8100e750d9eddde6410d2b8aa00fd0fbd1ada652 (CC @darstahl) appears to be assuming a behavior of os.Readlink that it does not actually provide, which will almost certainly cause Walk to fail when the root is a relative symlink or symlink-to-symlink on POSIX platforms.
A cleaner fix here is to ensure that the root path passed to filepath.Walk and/or filepath.WalkDir ends with a trailing separator, which (on all POSIX-compliant platforms, and Windows as of https://go.dev/cl/463177) will cause Walk to start walking at the target referred to by the root instead of the root symlink itself.
- (See https://go.dev/cl/484475 for an example of such a change.)
- Lenguaje dominante
- Go
- Estrellas
- 153
- Forks
- 74
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 containerd/continuity
-
Modernize repository Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
containerd/continuity#277 · 1 comentario ·
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
containerd/continuity#236 · 5 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
containerd/continuity#226 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
containerd/continuity#221 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
containerd/continuity#176 ·
Todos los issues de containerd/continuity
Issues similares
-
nix: vendorHash is outdated Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
Bob Shell support Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
santhosh-tekuri/jsonschema#276 ·