Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parse sing-box processPath into process #37

Merged
merged 1 commit into from
May 16, 2023
Merged

parse sing-box processPath into process #37

merged 1 commit into from
May 16, 2023

Conversation

Palatis
Copy link

@Palatis Palatis commented May 16, 2023

clash-api for sing-box use processPath field which is the full-path of the captured process.

  • On Windows: \Device\HarddiskVolume3\Path\To\Program.exe
  • On Linux: /path/to/program (username)

grab the last part of the full path and render as process.

clash-api for sing-box use `processPath` field which is the full-path of the captured process.
- On Windows: `\Device\HarddiskVolume3\Path\To\Program.exe`
- On Linux: `/path/to/program (username)`

grab the last part of the full path and render as `process`.
@Larvan2
Copy link

Larvan2 commented May 16, 2023

方便截个图看看效果吗?

@CHIZI-0618
Copy link

@Larvan2 应该是因为 sing-box 使用的 clash api 跟上游行为一致,metadata 只有 processPath,没有 process,所以 sing-box 在使用 Yacd-meta 时无法显示进程名,这个 pr 修复了

@Larvan2 Larvan2 merged commit e0cdaf9 into MetaCubeX:master May 16, 2023
@Palatis
Copy link
Author

Palatis commented May 16, 2023

Yacd-meta Process Names

you need at least one rule that does process name matching in sing-box config.json, or process names are ignored altogether.

Live GH Page

@Palatis Palatis deleted the sing-box/process_name branch May 17, 2023 16:00
d2184 pushed a commit to d2184/Yacd-meta that referenced this pull request Jun 17, 2023
clash-api for sing-box use `processPath` field which is the full-path of the captured process.
- On Windows: `\Device\HarddiskVolume3\Path\To\Program.exe`
- On Linux: `/path/to/program (username)`

grab the last part of the full path and render as `process`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants