fix: include Error in log message (#7955)
This addresses post-merge feedback from https://github.com/openai/codex/pull/7856.
This commit is contained in:
parent
b1905d3754
commit
7c18f7b680
1 changed files with 9 additions and 8 deletions
|
|
@ -425,14 +425,15 @@ impl SandboxPolicy {
|
|||
&& let Some(tmpdir) = std::env::var_os("TMPDIR")
|
||||
&& !tmpdir.is_empty()
|
||||
{
|
||||
if let Ok(tmpdir_path) =
|
||||
AbsolutePathBuf::from_absolute_path(PathBuf::from(&tmpdir))
|
||||
{
|
||||
roots.push(tmpdir_path);
|
||||
} else {
|
||||
error!(
|
||||
"Ignoring invalid TMPDIR value {tmpdir:?} for sandbox writable root",
|
||||
);
|
||||
match AbsolutePathBuf::from_absolute_path(PathBuf::from(&tmpdir)) {
|
||||
Ok(tmpdir_path) => {
|
||||
roots.push(tmpdir_path);
|
||||
}
|
||||
Err(e) => {
|
||||
error!(
|
||||
"Ignoring invalid TMPDIR value {tmpdir:?} for sandbox writable root: {e}",
|
||||
);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue