Validate the soft-delete suffix in /api/manage
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
Direzione di ricerca
Inizia da SoftDeleteRepo in csharp/src/HgResume.Api/Manage/RepoManageService.cs, dove Suffix da SoftDeleteRepoRequest (in Manage/ManageDtos.cs) viene passato a DeletedRepoName all'interno della chiamata a Directory.Move. Decidi se rifiutare il suffisso in anticipo con un 400 oppure verificare che il percorso risolto resti dentro la cartella eliminata, come suggerisce la issue. È completato quando i suffissi contenenti / o .. vengono rifiutati e il formato yyyy-MM-ddTHH-mm-ss__reset continua a funzionare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- C#
- Stelle
- 0
- Fork
- 1
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di sillsdev/hgresume
-
Difficoltà 3/5 Mezza giornata Idoneità per principianti 45/100
-
Difficoltà 3/5 Mezza giornata Idoneità per principianti 72/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 50/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Tutte le issue di sillsdev/hgresume
Issue simili
-
Importer: an empty or non-numeric Error cell is imported as 0 instead of missingForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Open-Systems-Pharmacology/OSPSuite.Core#3049 ·
I maintainer di solito rispondono entro 1 giorno
-
needs attention public-client untriaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
AzureAD/microsoft-authentication-library-for-dotnet#6242 ·
I maintainer di solito rispondono entro 2 giorni
-
docs: HTML config transform example sets advancedCustomization at a path the UI does not readAperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 90/100
criticalmanufacturing/cli#882 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Dome-Works/DOME#129 ·
I maintainer di solito rispondono entro 1 giorno
-
bug sev:L
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Systemorph/MeshWeaver#7480 ·
I maintainer di solito rispondono entro 1 giorno