chore: better getUpdateTime() for iterating all Geofiles (#1570)

component/updater/update_geo.go

@@ -229,20 +229,22 @@ func UpdateGeoDatabases() error {
 }
 
 func getUpdateTime() (err error, time time.Time) {
-	var fileInfo os.FileInfo
+	filesToCheck := []string{
-	if geodata.GeodataMode() {
+		C.Path.GeoIP(),
-		fileInfo, err = os.Stat(C.Path.GeoIP())
+		C.Path.MMDB(),
-		if err != nil {
+		C.Path.ASN(),
-			return err, time
+		C.Path.GeoSite(),
-		}
+	}
-	} else {
+
-		fileInfo, err = os.Stat(C.Path.MMDB())
+	for _, file := range filesToCheck {
-		if err != nil {
+		var fileInfo os.FileInfo
-			return err, time
+		fileInfo, err = os.Stat(file)
+		if err == nil {
+			return nil, fileInfo.ModTime()
 		}
 	}
 
-	return nil, fileInfo.ModTime()
+	return
 }
 
 func RegisterGeoUpdater() {

tunnel/tunnel.go

@@ -625,7 +625,7 @@ func match(metadata *C.Metadata) (C.Proxy, C.Rule, error) {
 				// normal check for process
 				uid, path, err := P.FindProcessName(metadata.NetWork.String(), metadata.SrcIP, int(metadata.SrcPort))
 				if err != nil {
-					log.Debugln("[Process] find process %s error: %v", metadata.String(), err)
+					log.Debugln("[Process] find process error for %s: %v", metadata.String(), err)
 				} else {
 					metadata.Process = filepath.Base(path)
 					metadata.ProcessPath = path
@@ -639,7 +639,7 @@ func match(metadata *C.Metadata) (C.Proxy, C.Rule, error) {
 				// check package names
 				pkg, err := P.FindPackageName(metadata)
 				if err != nil {
-					log.Debugln("[Process] find process %s error: %v", metadata.String(), err)
+					log.Debugln("[Process] find process error for %s: %v", metadata.String(), err)
 				} else {
 					metadata.Process = pkg
 				}