return error when mountoption is invalid. remove +x of generated prot…

…o file
buildbarn · Dec 14, 2024 · 8d24db8 · 8d24db8
1 parent f3fd780
commit 8d24db8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/pkg/filesystem/virtual/configuration/fuse_mount_enabled.go b/pkg/filesystem/virtual/configuration/fuse_mount_enabled.go
@@ -15,6 +15,8 @@ import (
 	"github.com/buildbarn/bb-storage/pkg/util"
 	go_fuse "github.com/hanwen/go-fuse/v2/fuse"
 	"github.com/jmespath/go-jmespath"
+	"google.golang.org/grpc/codes"
+	"google.golang.org/grpc/status"
 )
 
 func (m *fuseMount) Expose(terminationGroup program.Group, rootDirectory virtual.Directory) error {
@@ -72,7 +74,7 @@ func (m *fuseMount) Expose(terminationGroup program.Group, rootDirectory virtual
 	case virtualpb.FUSEMountConfiguration_FUSERMOUNT:
 		// No additional options needed for FUSERMOUNT
 	default:
-		// Default to use FUSERMOUNT
+		return status.Error(codes.InvalidArgument, "Invalid mount method")
 	}
 
 	server, err := go_fuse.NewServer(

diff --git a/pkg/proto/configuration/filesystem/virtual/virtual.pb.go b/pkg/proto/configuration/filesystem/virtual/virtual.pb.go