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

Feature process rule #372

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

lemos1235
Copy link
Contributor

add process rule

sample config:

[General]
dns-server = 223.5.5.5
tun = auto
loglevel = debug

[Proxy]
Direct = direct
SOCKS5 = socks,192.168.0.105,7890

[Rule]
# will match process path containing `curl`
PROCESS-NAME, curl, SOCKS5
# match process by pid
PROCESS-PID, 18868, SOCKS5
FINAL, Direct

@lemos1235 lemos1235 force-pushed the feature_process_rule branch 8 times, most recently from 49dc306 to a574c31 Compare April 7, 2023 01:52
@bdbai
Copy link
Contributor

bdbai commented Apr 13, 2023

  • Fetching process information is not always available in different environments such as iOS NE or UWP. What about making it an optional feature, and throw a warning whenever a config contains process rules?
  • PID is a volatile identifier. I am wondering whether PROCESS-PID rules are really useful...

@lemos1235
Copy link
Contributor Author

  • Fetching process information is not always available in different environments such as iOS NE or UWP. What about making it an optional feature, and throw a warning whenever a config contains process rules?
  • PID is a volatile identifier. I am wondering whether PROCESS-PID rules are really useful...
  • I will add a warning log later if the rule is not supported
  • Find the PID (Process ID) of Telegram, then configure the rule to observe if it can use the proxy.

@bdbai
Copy link
Contributor

bdbai commented Apr 13, 2023

Find the PID (Process ID) of Telegram, then configure the rule to observe if it can use the proxy.

I think there is a built-in connectivity indicator in the Telegram client? Anyway, users tweaking proxies can always comment out all rules but the FINAL one.

@lemos1235 lemos1235 force-pushed the feature_process_rule branch 2 times, most recently from e9c13c2 to 934ee8f Compare April 15, 2023 14:36
@lemos1235 lemos1235 force-pushed the feature_process_rule branch 2 times, most recently from 9a8d15f to b69957e Compare March 23, 2024 01:53
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.

2 participants