Fix wrong process id format (#215)

mrproliu · web-flow · commit 6e2413a14af7 · 2025-04-21T12:15:27.000+08:00
diff --git a/CHANGES.md b/CHANGES.md
@@ -15,6 +15,8 @@ Release Notes.
 
 ### Bug Fixes
 
+* Fix wrong process id format by @mrproliu in https://github.com/apache/skywalking-cli/pull/215
+
 0.14.0
 ------------------
 
diff --git a/internal/commands/interceptor/process.go b/internal/commands/interceptor/process.go
@@ -83,7 +83,7 @@ func parseProcess(required bool, idFlagName, nameFlagName, instanceIDFlagName st
 			if instanceID == "" {
 				return fmt.Errorf(`"--%s" is specified but its related service name or id is not given`, nameFlagName)
 			}
-			id = fmt.Sprintf("%x", sha256.Sum256([]byte(fmt.Sprintf("%s_%s", instanceID, name))))
+			id = fmt.Sprintf("%x", sha256.Sum256([]byte(fmt.Sprintf("%s_%s", name, instanceID))))
 		}
 
 		return ctx.Set(idFlagName, id)

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ func parseProcess(required bool, idFlagName, nameFlagName, instanceIDFlagName st`
`83`	`83`	`if instanceID == "" {`
`84`	`84`	return fmt.Errorf(`"--%s" is specified but its related service name or id is not given`, nameFlagName)
`85`	`85`	`}`
`86`		`- id = fmt.Sprintf("%x", sha256.Sum256([]byte(fmt.Sprintf("%s_%s", instanceID, name))))`
	`86`	`+ id = fmt.Sprintf("%x", sha256.Sum256([]byte(fmt.Sprintf("%s_%s", name, instanceID))))`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`return ctx.Set(idFlagName, id)`