avoid unknown actions

backend/service/client.go

@@ -6,6 +6,7 @@ import (
 	"s-ui/database/model"
 	"s-ui/logger"
 	"s-ui/util"
+	"s-ui/util/common"
 	"time"
 
 	"gorm.io/gorm"
@@ -85,6 +86,8 @@ func (s *ClientService) Save(tx *gorm.DB, act string, data json.RawMessage, host
 		if err != nil {
 			return nil, err
 		}
+	default:
+		return nil, common.NewErrorf("unknown action: %s", act)
 	}
 
 	return inboundIds, nil

backend/service/endpoints.go

@@ -5,6 +5,7 @@ import (
 	"os"
 	"s-ui/database"
 	"s-ui/database/model"
+	"s-ui/util/common"
 
 	"gorm.io/gorm"
 )
@@ -56,10 +57,10 @@ func (o *EndpointService) GetAllConfig(db *gorm.DB) ([]json.RawMessage, error) {
 	return endpointsJson, nil
 }
 
-func (s *EndpointService) Save(tx *gorm.DB, action string, data json.RawMessage) error {
+func (s *EndpointService) Save(tx *gorm.DB, act string, data json.RawMessage) error {
 	var err error
 
-	switch action {
+	switch act {
 	case "new", "edit":
 		var endpoint model.Endpoint
 		err = endpoint.UnmarshalJSON(data)
@@ -72,7 +73,7 @@ func (s *EndpointService) Save(tx *gorm.DB, action string, data json.RawMessage)
 			if err != nil {
 				return err
 			}
-			if action == "edit" {
+			if act == "edit" {
 				var oldTag string
 				err = tx.Model(model.Endpoint{}).Select("tag").Where("id = ?", endpoint.Id).Find(&oldTag).Error
 				if err != nil {
@@ -109,6 +110,8 @@ func (s *EndpointService) Save(tx *gorm.DB, action string, data json.RawMessage)
 		if err != nil {
 			return err
 		}
+	default:
+		return common.NewErrorf("unknown action: %s", act)
 	}
 	return nil
 }

backend/service/inbounds.go

@@ -6,6 +6,7 @@ import (
 	"s-ui/database"
 	"s-ui/database/model"
 	"s-ui/util"
+	"s-ui/util/common"
 	"strings"
 
 	"gorm.io/gorm"
@@ -153,6 +154,8 @@ func (s *InboundService) Save(tx *gorm.DB, act string, data json.RawMessage, hos
 		if err != nil {
 			return 0, err
 		}
+	default:
+		return 0, common.NewErrorf("unknown action: %s", act)
 	}
 	return id, nil
 }

backend/service/outbounds.go

@@ -5,6 +5,7 @@ import (
 	"os"
 	"s-ui/database"
 	"s-ui/database/model"
+	"s-ui/util/common"
 
 	"gorm.io/gorm"
 )
@@ -56,10 +57,10 @@ func (o *OutboundService) GetAllConfig(db *gorm.DB) ([]json.RawMessage, error) {
 	return outboundsJson, nil
 }
 
-func (s *OutboundService) Save(tx *gorm.DB, action string, data json.RawMessage) error {
+func (s *OutboundService) Save(tx *gorm.DB, act string, data json.RawMessage) error {
 	var err error
 
-	switch action {
+	switch act {
 	case "new", "edit":
 		var outbound model.Outbound
 		err = outbound.UnmarshalJSON(data)
@@ -72,7 +73,7 @@ func (s *OutboundService) Save(tx *gorm.DB, action string, data json.RawMessage)
 			if err != nil {
 				return err
 			}
-			if action == "edit" {
+			if act == "edit" {
 				var oldTag string
 				err = tx.Model(model.Outbound{}).Select("tag").Where("id = ?", outbound.Id).Find(&oldTag).Error
 				if err != nil {
@@ -109,6 +110,8 @@ func (s *OutboundService) Save(tx *gorm.DB, action string, data json.RawMessage)
 		if err != nil {
 			return err
 		}
+	default:
+		return common.NewErrorf("unknown action: %s", act)
 	}
 	return nil
 }