From 16376612f50b04543733f69630b36dbaeb7d7a70 Mon Sep 17 00:00:00 2001
From: iseki <admin@iseki.space>
Date: Mon, 21 Oct 2024 16:52:07 +0800
Subject: [PATCH] refactor: remove ctx in struct

---
 cmd/murphy/internal/internalcmd/scanner_scan.go | 1 -
 cmd/murphy/internal/scan/sbomscan.go            | 1 -
 cmd/murphy/internal/scan/scan.go                | 2 --
 model/inspection_task.go                        | 9 +--------
 model/scantask.go                               | 2 --
 5 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/cmd/murphy/internal/internalcmd/scanner_scan.go b/cmd/murphy/internal/internalcmd/scanner_scan.go
index 9f8f8d4f..a3c55720 100644
--- a/cmd/murphy/internal/internalcmd/scanner_scan.go
+++ b/cmd/murphy/internal/internalcmd/scanner_scan.go
@@ -53,7 +53,6 @@ func scannerScanRun(cmd *cobra.Command, args []string) {
 	}
 
 	var scantask = &model.ScanTask{
-		Ctx:         ctx,
 		ProjectPath: scanDir,
 		AccessType:  model.AccessTypeCli,
 		Mode:        model.ScanModeSource,
diff --git a/cmd/murphy/internal/scan/sbomscan.go b/cmd/murphy/internal/scan/sbomscan.go
index b104e574..9653773d 100644
--- a/cmd/murphy/internal/scan/sbomscan.go
+++ b/cmd/murphy/internal/scan/sbomscan.go
@@ -57,7 +57,6 @@ func SbomScan() *cobra.Command {
 func processDir(ctx context.Context, dir string) ([]byte, error) {
 	var e error
 	var task = model.ScanTask{
-		Ctx:         ctx,
 		ProjectPath: dir,
 	}
 	ctx = model.WithScanTask(ctx, &task)
diff --git a/cmd/murphy/internal/scan/scan.go b/cmd/murphy/internal/scan/scan.go
index 16add181..f071773b 100644
--- a/cmd/murphy/internal/scan/scan.go
+++ b/cmd/murphy/internal/scan/scan.go
@@ -62,7 +62,6 @@ func envScan(ctx context.Context) (task *model.ScanTask, e error) {
 	cv.DisplaySubtaskCreated(ctx, createTaskResp.ProjectsName, createTaskResp.SubtaskID)
 	// create task object
 	task = &model.ScanTask{
-		Ctx:         ctx,
 		Mode:        createSubtask.ScanMode,
 		AccessType:  createSubtask.AccessType,
 		TaskId:      createTaskResp.TaskID,
@@ -175,7 +174,6 @@ func scan(ctx context.Context, dir string, accessType model.AccessType, mode mod
 
 	// create task object
 	task := &model.ScanTask{
-		Ctx:             ctx,
 		Mode:            mode,
 		AccessType:      accessType,
 		ProjectPath:     dir,
diff --git a/model/inspection_task.go b/model/inspection_task.go
index 8dd0d639..1f913a96 100644
--- a/model/inspection_task.go
+++ b/model/inspection_task.go
@@ -3,8 +3,6 @@ package model
 import (
 	"context"
 	"path/filepath"
-
-	"github.com/murphysecurity/murphysec/infra/logctx"
 )
 
 type InspectionTask struct {
@@ -49,13 +47,8 @@ func UseInspectionTask(ctx context.Context) *InspectionTask {
 }
 
 func (i *InspectionTask) AddModule(module Module) {
-	var logger = logctx.Use(i.ctx).Sugar()
-	logger.Infof("add module: %v", module)
 	if filepath.IsAbs(module.ModulePath) {
-		relPath, e := filepath.Rel(i.scanTask.ProjectPath, module.ModulePath)
-		if e != nil {
-			logger.Warnf("get module relative-path: %v", e)
-		}
+		relPath, _ := filepath.Rel(i.scanTask.ProjectPath, module.ModulePath)
 		module.ModulePath = relPath
 	}
 	if module.ModulePath == "." {
diff --git a/model/scantask.go b/model/scantask.go
index 9cb9c7f3..792c3ad5 100644
--- a/model/scantask.go
+++ b/model/scantask.go
@@ -6,7 +6,6 @@ import (
 
 // ScanTask 表示当前扫描任务
 type ScanTask struct {
-	Ctx             context.Context
 	ProjectPath     string
 	AccessType      AccessType
 	Mode            ScanMode
@@ -24,7 +23,6 @@ type ScanTask struct {
 
 func (s *ScanTask) BuildInspectionTask(dir string) *InspectionTask {
 	return &InspectionTask{
-		ctx:           s.Ctx,
 		scanTask:      s,
 		inspectionDir: dir,
 	}