small fix: UI now disable endpoint controls correct when endpoint configured on the server (in config.json)

Author: Amper

cmd/sql-to-logsql/api/server_test.go

@@ -367,11 +367,8 @@ func TestHandleDescribeWithoutEndpoint(t *testing.T) {
 	rr := httptest.NewRecorder()
 	srv.ServeHTTP(rr, req)
 
-	if rr.Code != http.StatusBadRequest {
-		t.Fatalf("expected status 400, got %d", rr.Code)
-	}
-	if !strings.Contains(rr.Body.String(), "endpoint is required for this statement") {
-		t.Fatalf("unexpected error message: %s", rr.Body.String())
+	if rr.Code != http.StatusOK {
+		t.Fatalf("expected status 200, got %d", rr.Code)
 	}
 }
 

cmd/sql-to-logsql/web/ui/src/components/logs-endpoint/LogsEndpoint.tsx

@@ -34,7 +34,7 @@ export function LogsEndpoint({
       <CardHeader>
         <CardTitle>VictoriaLogs endpoint</CardTitle>
         {!endpointReadOnly && <CardDescription>You can query data from VictoriaLogs instance or just translate SQl to LogsQL without querying</CardDescription>}
-        <CardAction className={"flex flex-row gap-2"}>
+        {!endpointReadOnly && <CardAction className={"flex flex-row gap-2"}>
           <Switch
               checked={endpointEnabled}
               id={"endpointEnabled"}
@@ -46,7 +46,7 @@ export function LogsEndpoint({
           <Label htmlFor={"endpointEnabled"} className={"cursor-pointer"}>
               Query data from VictoriaLogs
           </Label>
-        </CardAction>
+        </CardAction>}
       </CardHeader>
       <CardContent className={"flex flex-row gap-2"}>
         <div className={"flex flex-col gap-1 w-3/4"}>

cmd/sql-to-logsql/web/ui/src/components/query-results/QueryResults.tsx

@@ -37,7 +37,7 @@ export function QueryResults({ query, data, isLoading, endpointEnabled }: QueryR
           </CardDescription>
         }
       </CardHeader>
-      {endpointEnabled &&
+      {!!data &&
         <CardContent>
           <QueryResultsTable data={data} />
         </CardContent>

lib/vlogs/api.go

@@ -46,8 +46,14 @@ func (a *API) Execute(ctx context.Context, si *logsql.StatementInfo, customEC En
 	}
 	switch si.Kind {
 	case logsql.StatementTypeSelect:
+		if a.ec.Endpoint == "" && customEC.Endpoint == "" {
+			return nil, nil
+		}
 		return a.Query(ctx, si.LogsQL, recEC)
 	case logsql.StatementTypeDescribe:
+		if a.ec.Endpoint == "" && customEC.Endpoint == "" {
+			return nil, nil
+		}
 		return a.GetFieldNames(ctx, si.LogsQL, recEC)
 	case logsql.StatementTypeCreateView, logsql.StatementTypeDropView:
 		return []byte(si.Data), nil