fixing privileges; checking flags on start up

cockroachlabs · Nov 18, 2022 · cd92fca · cd92fca
1 parent f16d9d0
commit cd92fca
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/internal/cmd/server/server.go b/internal/cmd/server/server.go
@@ -16,6 +16,7 @@
 package server
 
 import (
+	"errors"
 	"os"
 	"os/signal"
 	"syscall"
@@ -41,6 +42,12 @@ func Command() *cobra.Command {
 ./visus start --bindAddr "127.0.0.1:15432" `,
 		RunE: func(cmd *cobra.Command, _ []string) error {
 			ctx := cmd.Context()
+			if (cfg.BindCert == "" || cfg.BindKey == "") && !cfg.Insecure {
+				return errors.New("--insecure must be specfied if certificates and private key are missing")
+			}
+			if cfg.URL == "" {
+				return errors.New("--url must be specified")
+			}
 			conn, err := database.DefaultFactory.New(ctx, cfg.URL)
 			if err != nil {
 				return err

diff --git a/internal/store/sql/ddl.sql b/internal/store/sql/ddl.sql
@@ -6,7 +6,7 @@ GRANT CONNECT ON DATABASE _visus to visus;
 
 USE _visus;
 
-ALTER DEFAULT PRIVILEGES GRANT SELECT ON TABLES TO visus_monitor;
+ALTER DEFAULT PRIVILEGES GRANT SELECT ON TABLES TO visus;
 
 CREATE TYPE IF NOT EXISTS _visus.scope AS ENUM ('node', 'cluster');