Add tests for resolveSafePath
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 88/100
Direzione di ricerca
Leggi resolveSafePath alla fine di internal/tools/files.go, quindi esamina le convenzioni esistenti in internal/tools/tools_test.go. Aggiungi casi per i percorsi indicati usando una radice temporanea ed esegui i test del pacchetto tools. Il lavoro è completato quando i percorsi all'interno della radice vengono risolti senza errori e ogni percorso che ne esce o è assoluto restituisce un errore.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What
resolveSafePath is the function that jails every file tool to the launch directory. It is pretty thin right now, add tests to try and break it and find real work around bugs to fix.
Where
internal/tools/files.go — the function is at the bottom of the file.
Tests go in internal/tools/tools_test.go (same package tools, so unexported funcs are reachable).
Cases to cover
Should resolve (no error), given a root like /tmp/root:
"main.go"→ inside root"sub/dir/file.go"→ inside root"./main.go"→ inside root (cleanup)""→ root itself
Should error (path escapes root):
"../secret""../../etc/passwd""sub/../../escape"- an absolute path like
"/etc/passwd"(should not silently land inside root)
Notes
It's a pure function do not need ollama, try and break it its good if you find it failing
- Lingua principale
- Go
- Stelle
- 1
- Fork
- 2
- Merge medio
- 1h 46m
- PR unite (30g)
- 6
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la 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 Edwin-metcalf/MetCode
-
enhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Edwin-metcalf/MetCode#16 ·
-
Add a MakefileApertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Edwin-metcalf/MetCode#8 ·
-
Add a --host flagApertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
Edwin-metcalf/MetCode#7 ·
-
Add a --version flagApertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Edwin-metcalf/MetCode#6 ·
-
security
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Edwin-metcalf/MetCode#17 ·
Tutte le issue di Edwin-metcalf/MetCode
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
SAP/crossplane-provider-btp#1118 ·
I maintainer di solito rispondono entro 1 giorno
-
benchmark: `make setup` fails because audit-exporter builds with Go 1.25.0 while go.mod requires 1.26.0Forse già presa @xsvm l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
volcano-sh/volcano#6097 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
os: RawConn methods of a closed File return internal/poll.ErrFileClosing instead of ErrClosedAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Bedrock invoke-with-response-stream reports `stop_reason: "end_turn"` for tool-use turnsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno