You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by li...@apache.org on 2025/04/21 04:05:53 UTC
(skywalking-cli) 01/01: Fix wrong process id format
This is an automated email from the ASF dual-hosted git repository.
liuhan pushed a commit to branch fix-process-id-format
in repository https://gitbox.apache.org/repos/asf/skywalking-cli.git
commit 88567cc173d01aa0aa96df9d0d1963184b276e33
Author: mrproliu <74...@qq.com>
AuthorDate: Mon Apr 21 12:05:41 2025 +0800
Fix wrong process id format
---
CHANGES.md | 2 ++
internal/commands/interceptor/process.go | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/CHANGES.md b/CHANGES.md
index fa7d53e..70bed05 100644
--- 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
index 9fbc7f2..fd80855 100644
--- 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)