Update logic to get socket path

* Have same fallbacks as the official client
* Reduce number of allocations

src/connection/base.rs

@@ -1,5 +1,5 @@
 use std::{
-    io::{Write, Read, ErrorKind},
+    io::{ErrorKind, Read, Write},
     marker::Sized,
     path::PathBuf,
     thread,
@@ -44,7 +44,9 @@ pub trait Connection: Sized {
 
     /// The full socket path.
     fn socket_path(n: u8) -> PathBuf {
-        Self::ipc_path().join(format!("discord-ipc-{}", n))
+        let mut path = Self::ipc_path();
+        path.push(format!("discord-ipc-{}", n));
+        path
     }
 
     /// Perform a handshake on this socket connection.

src/connection/unix.rs

@@ -1,9 +1,9 @@
 use std::{
-    time,
-    path::PathBuf,
     env,
-    os::unix::net::UnixStream,
     net::Shutdown,
+    os::unix::net::UnixStream,
+    path::PathBuf,
+    time
 };
 use super::base::Connection;
 use crate::Result;
@@ -28,12 +28,8 @@ impl Connection for UnixConnection {
     fn ipc_path() -> PathBuf {
         let tmp = env::var("XDG_RUNTIME_DIR")
             .or_else(|_| env::var("TMPDIR"))
-            .or_else(|_| {
-                match env::temp_dir().to_str() {
-                    None => Err("Failed to convert temp_dir"),
-                    Some(tmp) => Ok(tmp.to_owned())
-                }
-            })
+            .or_else(|_| env::var("TMP"))
+            .or_else(|_| env::var("TEMP"))
             .unwrap_or_else(|_| "/tmp".to_owned());
         PathBuf::from(tmp)
     }