Skip to content

Commit 7971f28

Browse files
authored
Merge pull request #79 from Romankivs/fix-namespace-hardcoded-display
fix: use actual $DISPLAY value when detemining socket path's
2 parents 5bdc021 + 64256fe commit 7971f28

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

crates/ninep/src/server.rs

+10-4
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,22 @@ impl Drop for Socket {
3030
}
3131
}
3232

33+
const DEFAULT_DISPLAY_VALUE: &str = ":0";
34+
35+
fn socket_dir() -> String {
36+
let uname = env::var("USER").unwrap();
37+
let display = env::var("DISPLAY").unwrap_or(String::from(DEFAULT_DISPLAY_VALUE));
38+
format!("/tmp/ns.{uname}.{display}")
39+
}
40+
3341
/// The unix socket path that will be used for a given server name.
3442
pub fn socket_path(name: &str) -> String {
35-
let uname = env::var("USER").unwrap();
36-
let socket_dir = format!("/tmp/ns.{uname}.:0");
43+
let socket_dir = socket_dir();
3744
format!("{socket_dir}/{name}")
3845
}
3946

4047
fn unix_socket(name: &str) -> Socket {
41-
let uname = env::var("USER").unwrap();
42-
let socket_dir = format!("/tmp/ns.{uname}.:0");
48+
let socket_dir = socket_dir();
4349
let _ = fs::create_dir_all(&socket_dir);
4450
let path = format!("{socket_dir}/{name}");
4551

0 commit comments

Comments
 (0)