sockets: make NewUnixSocket, WithChown, WithChmod unix-only

The WithChown and WithChmod options are not supported on Windows.
We could provide stubs in future, but let's start with not
including them.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Author: thaJeztah

sockets/unix_socket.go

@@ -55,26 +55,6 @@ import (
 // SockOption sets up socket file's creating option
 type SockOption func(string) error
 
-// WithChown modifies the socket file's uid and gid
-func WithChown(uid, gid int) SockOption {
-	return func(path string) error {
-		if err := os.Chown(path, uid, gid); err != nil {
-			return err
-		}
-		return nil
-	}
-}
-
-// WithChmod modifies socket file's access mode.
-func WithChmod(mask os.FileMode) SockOption {
-	return func(path string) error {
-		if err := os.Chmod(path, mask); err != nil {
-			return err
-		}
-		return nil
-	}
-}
-
 // NewUnixSocketWithOpts creates a unix socket with the specified options.
 // By default, socket permissions are 0000 (i.e.: no access for anyone); pass
 // WithChmod() and WithChown() to set the desired ownership and permissions.
@@ -102,8 +82,3 @@ func NewUnixSocketWithOpts(path string, opts ...SockOption) (net.Listener, error
 
 	return l, nil
 }
-
-// NewUnixSocket creates a unix socket with the specified path and group.
-func NewUnixSocket(path string, gid int) (net.Listener, error) {
-	return NewUnixSocketWithOpts(path, WithChown(0, gid), WithChmod(0o660))
-}

sockets/unix_socket_test.go

@@ -32,22 +32,6 @@ func runTest(t *testing.T, path string, l net.Listener, echoStr string) {
 	}
 }
 
-// TestNewUnixSocket run under root user.
-func TestNewUnixSocket(t *testing.T) {
-	if os.Getuid() != 0 {
-		t.Skip("requires root")
-	}
-	gid := os.Getgid()
-	path := "/tmp/test.sock"
-	echoStr := "hello"
-	l, err := NewUnixSocket(path, gid)
-	if err != nil {
-		t.Fatal(err)
-	}
-	defer func() { _ = l.Close() }()
-	runTest(t, path, l, echoStr)
-}
-
 func TestUnixSocketWithOpts(t *testing.T) {
 	socketFile, err := os.CreateTemp("", "test*.sock")
 	if err != nil {

sockets/unix_socket_unix.go

@@ -4,9 +4,35 @@ package sockets
 
 import (
 	"net"
+	"os"
 	"syscall"
 )
 
+// WithChown modifies the socket file's uid and gid
+func WithChown(uid, gid int) SockOption {
+	return func(path string) error {
+		if err := os.Chown(path, uid, gid); err != nil {
+			return err
+		}
+		return nil
+	}
+}
+
+// WithChmod modifies socket file's access mode.
+func WithChmod(mask os.FileMode) SockOption {
+	return func(path string) error {
+		if err := os.Chmod(path, mask); err != nil {
+			return err
+		}
+		return nil
+	}
+}
+
+// NewUnixSocket creates a unix socket with the specified path and group.
+func NewUnixSocket(path string, gid int) (net.Listener, error) {
+	return NewUnixSocketWithOpts(path, WithChown(0, gid), WithChmod(0o660))
+}
+
 func listenUnix(path string) (net.Listener, error) {
 	// net.Listen does not allow for permissions to be set. As a result, when
 	// specifying custom permissions ("WithChmod()"), there is a short time

sockets/unix_socket_unix_test.go

@@ -30,3 +30,19 @@ func createTestUnixSocket(t *testing.T, path string) (listener net.Listener) {
 	}
 	return l
 }
+
+// TestNewUnixSocket run under root user.
+func TestNewUnixSocket(t *testing.T) {
+	if os.Getuid() != 0 {
+		t.Skip("requires root")
+	}
+	gid := os.Getgid()
+	path := "/tmp/test.sock"
+	echoStr := "hello"
+	l, err := NewUnixSocket(path, gid)
+	if err != nil {
+		t.Fatal(err)
+	}
+	defer func() { _ = l.Close() }()
+	runTest(t, path, l, echoStr)
+}