Validate the soft-delete suffix in /api/manage
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Empieza en SoftDeleteRepo en csharp/src/HgResume.Api/Manage/RepoManageService.cs, donde Suffix de SoftDeleteRepoRequest (en Manage/ManageDtos.cs) se pasa a DeletedRepoName dentro de la llamada a Directory.Move. Decide si rechazar el sufijo de antemano con un 400 o comprobar que la ruta resuelta se mantenga dentro de la carpeta eliminada, como sugiere el issue. Está terminado cuando se rechacen los sufijos que contengan / o .. y la forma yyyy-MM-ddTHH-mm-ss__reset siga funcionando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
POST /api/manage/repos/{code}/soft-delete passes SoftDeleteRepoRequest.Suffix straight into the destination path, with no validation beyond [Required, MinLength(1)]:
// RepoManageService.SoftDeleteRepo
Directory.Move(
PrefixRepoFilePath(code),
Path.Combine(deletedRepoPath, DeletedRepoName(code, deletedRepoSuffix))); // $"{code}__{suffix}"
A suffix that contains / or .. segments (e.g. x/../../../somewhere) moves the repo outside _____deleted_____. A suffix like a/b fails with a 500 because the intermediate directory doesn't exist.
The X-Manage-Secret check means only a trusted caller can do this, so the impact is low. It is still worth hardening, since the suffix is the only part of the path the caller controls freely.
Suggested fix: validate the suffix the same way as project codes, e.g. restrict it to [A-Za-z0-9_-] (which still allows the yyyy-MM-ddTHH-mm-ss__reset timestamps LexBox uses) and return 400 otherwise. Alternatively, check that the resolved full path is still under the deleted folder.
Source: csharp/src/HgResume.Api/Manage/RepoManageService.cs (SoftDeleteRepo), Manage/ManageDtos.cs (SoftDeleteRepoRequest).
- Lenguaje dominante
- C#
- Estrellas
- 0
- Forks
- 1
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin 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 sillsdev/hgresume
-
Dificultad 3/5 Medio día Aptitud para principiantes 45/100
-
Dificultad 3/5 Medio día Aptitud para principiantes 72/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 50/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de sillsdev/hgresume
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
:watch: Not Triaged advanced-concepts/subsvc dotnet-csharp/svc
Dificultad 1/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
[bug] Season images use the season's TMDB id as a TVDB series idPosiblemente ocupada @ViKodex la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
enhancement no-stack-trace
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
greenshot/greenshot#1618 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
microsoft/aspire.dev#1962 ·
Los mantenedores suelen responder en 1 día