aljaz/goose
1
0
Fork 0
mirror of https://github.com/aljazceru/goose.git synced 2025-12-18 14:44:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

allow to use dev/null for no-session mode (#3176)

Browse Source
This commit is contained in:
Max Novich
2025-06-30 13:37:33 -07:00
committed by GitHub
parent 517f03fff3
commit b9b7ed5a6a
1 changed files with 16 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
crates/goose/src/session/storage.rs
View File

@@ -158,6 +158,14 @@ pub fn get_path(id: Identifier) -> Result<PathBuf> {
session_dir.join(format!("{}.jsonl", name))
}
Identifier::Path(path) => {
// Allow special paths for no-session mode
if let Some(path_str) = path.to_str() {
if path_str == "/dev/null" || path_str == "NUL" {
// These are special paths used for --no-session mode
return Ok(path);
}
}
// In test mode, allow temporary directory paths
#[cfg(test)]
{
@@ -190,10 +198,14 @@ pub fn get_path(id: Identifier) -> Result<PathBuf> {
}
};
// Additional security check for file extension
if let Some(ext) = path.extension() {
if ext != "jsonl" {
return Err(anyhow::anyhow!("Invalid file extension"));
// Additional security check for file extension (skip for special no-session paths)
if let Some(path_str) = path.to_str() {
if path_str != "/dev/null" && path_str != "NUL" {
if let Some(ext) = path.extension() {
if ext != "jsonl" {
return Err(anyhow::anyhow!("Invalid file extension"));
}
}
}
}