Refactor WAF service methods to use UUID instead of ID for attack log retrieval

- Update GetAttackLogDetails and GetAttackLogRsp methods in the WafController and wafService to accept UUID as a string parameter instead of a uint32 ID.
- Modify corresponding service interface to reflect the change in parameter type for improved clarity and consistency in handling attack logs.

internal/controller/waf.go

@@ -375,14 +375,9 @@ func (c *WafController) GetAttackLogDetails(ctx *gin.Context) {
 	ctx1, cancel := context.WithTimeout(context.Background(), 10*time.Second)
 	defer cancel()
 
-	id := ctx.Query("uuid")
-	idUint, err := strconv.ParseUint(id, 10, 32)
-	if err != nil {
-		utils.AssembleResponse(ctx, nil, err)
-		return
-	}
+	uuid := ctx.Query("uuid")
 
-	details, err := c.service.GetAttackLogDetails(ctx1, uint32(idUint))
+	details, err := c.service.GetAttackLogDetails(ctx1, uuid)
 	if err != nil {
 		utils.AssembleResponse(ctx, nil, err)
 		return
@@ -397,12 +392,8 @@ func (c *WafController) GetAttackLogRsp(ctx *gin.Context) {
 	ctx1, cancel := context.WithTimeout(context.Background(), 10*time.Second)
 	defer cancel()
 
-	id := ctx.Query("uuid")
-	idUint, err := strconv.ParseUint(id, 10, 32)
-	if err != nil {
-		utils.AssembleResponse(ctx, nil, err)
-		return
-	}
+	uuid := ctx.Query("uuid")
+
 	length := ctx.Query("length")
 	lengthUint, err := strconv.ParseUint(length, 10, 32)
 	if err != nil {
@@ -410,7 +401,7 @@ func (c *WafController) GetAttackLogRsp(ctx *gin.Context) {
 		return
 	}
 
-	rsp, err := c.service.GetAttackLogRsp(ctx1, uint32(idUint), uint32(lengthUint))
+	rsp, err := c.service.GetAttackLogRsp(ctx1, uuid, uint32(lengthUint))
 	if err != nil {
 		utils.AssembleResponse(ctx, nil, err)
 		return

internal/service/service.go

@@ -20,8 +20,8 @@ type Service interface {
 	DeleteListenerWaf(ctx context.Context, req *DeleteListenerReq) error
 	EnableListenerWafs(ctx context.Context, req *EnableListenerWafsReq) error
 	ListAttackLogs(ctx context.Context, req *AttackLogFilter) ([]AttackLog, string, error)
-	GetAttackLogDetails(ctx context.Context, id uint32) (*AttackLog, error)
-	GetAttackLogRsp(ctx context.Context, id uint32, length uint32) (*AttackRsp, error)
+	GetAttackLogDetails(ctx context.Context, uuid string) (*AttackLog, error)
+	GetAttackLogRsp(ctx context.Context, uuid string, length uint32) (*AttackRsp, error)
 	ListRules(ctx context.Context, regionCode, namespace, gatewayName, language, name string) ([]RuleGroupResp, error)
 	CreateBlackWhiteList(ctx context.Context, req *MatcherExpr) error
 	UpdateBlackWhiteList(ctx context.Context, req *MatcherExpr) error

internal/service/waf.go

@@ -1004,9 +1004,9 @@ func (s *wafService) ListAttackLogs(ctx context.Context, req *AttackLogFilter) (
 	return attackLogs, pageToken, nil
 }
 
-func (s *wafService) GetAttackLogDetails(ctx context.Context, id uint32) (*AttackLog, error) {
+func (s *wafService) GetAttackLogDetails(ctx context.Context, uuid string) (*AttackLog, error) {
 	res, err := s.elasticClient.Search("waf-detections*").
-		Query(elastic.NewTermQuery("id.keyword", id)).Do(ctx)
+		Query(elastic.NewTermQuery("id.keyword", uuid)).Do(ctx)
 	if err != nil {
 		return nil, fmt.Errorf("failed to search waf detections: %v", err)
 	}
@@ -1030,9 +1030,9 @@ func (s *wafService) GetAttackLogDetails(ctx context.Context, id uint32) (*Attac
 	return attackLog, nil
 }
 
-func (s *wafService) GetAttackLogRsp(ctx context.Context, id uint32, length uint32) (*AttackRsp, error) {
+func (s *wafService) GetAttackLogRsp(ctx context.Context, uuid string, length uint32) (*AttackRsp, error) {
 	res, err := s.elasticClient.Search("waf-detections*").
-		Query(elastic.NewTermQuery("id.keyword", id)).Do(ctx)
+		Query(elastic.NewTermQuery("id.keyword", uuid)).Do(ctx)
 	if err != nil {
 		return nil, fmt.Errorf("failed to search waf detections: %v", err)
 	}