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.

Sign up for GitHub

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

Jump to bottom

Multiple instances of ss_privoxy #665

Merged
merged 3 commits into from
Aug 18, 2016
Merged

Multiple instances of ss_privoxy #665

merged 3 commits into from
Aug 18, 2016

Conversation

Noisyfox
Copy link
Contributor

No description provided.

@Noisyfox
Use job object to manage polipo service's external processes.
0162a57 
Signed-off-by: noisyfox <timemanager.rick@gmail.com>
@Noisyfox
Only kill those ss_privoxy.exe created by this ss instance.
161e136 
Signed-off-by: noisyfox <timemanager.rick@gmail.com>
@Noisyfox
Update README
6b2569e 
Signed-off-by: noisyfox <timemanager.rick@gmail.com>
@chenshaoju
Copy link
Collaborator

Quick test passed on appveyor version.
👍 Looks good to me, but someone else must approve.

@wongsyrone wongsyrone merged commit a1eef6e into shadowsocks:master Aug 18, 2016
@wongsyrone
Copy link
Contributor

Merged, thanks.

@Noisyfox Noisyfox deleted the dev_multi_privoxy branch August 18, 2016 15:38
@Noisyfox
Copy link
Contributor Author

The Chinese wiki need to update too.

@wongsyrone
Copy link
Contributor

Done.

@wongsyrone wongsyrone mentioned this pull request Aug 21, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Noisyfox @chenshaoju @wongsyrone