Skip to content

Pool #694

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

Merged
merged 17 commits into from
Nov 7, 2021
Merged

Pool #694

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ea59e9b
Refactor pool
abhinavsingh Nov 6, 2021
332583f
mypy fixes
abhinavsingh Nov 6, 2021
820ce46
Fix import (relative)
abhinavsingh Nov 7, 2021
8cbe1e9
Add WebScraper example skeleton & ConnectionPool skeleton
abhinavsingh Nov 7, 2021
d65b32c
Add ConnectionPool class
abhinavsingh Nov 7, 2021
8047f6b
Integrate ConnectionPool with proxy server (experimental)
abhinavsingh Nov 7, 2021
7e385c8
Lint fixes
abhinavsingh Nov 7, 2021
c5c2040
Remove unused imports. TODO: Put pool behind a flag. Default to false…
abhinavsingh Nov 7, 2021
ea4a0c0
Make ConnectionPool multiprocess safe. Later we want to make it safe…
abhinavsingh Nov 7, 2021
b36a59b
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Nov 7, 2021
2d5b477
Remove unused imports
abhinavsingh Nov 7, 2021
92edb70
Merge branch 'workpool' of github.com:abhinavsingh/proxy.py into work…
abhinavsingh Nov 7, 2021
2ccdcb2
Return created flag from acquire
abhinavsingh Nov 7, 2021
a785b87
Guard connection pool behind --enable-conn-pool flag
abhinavsingh Nov 7, 2021
a9d9c12
Flag belongs within connection pool class
abhinavsingh Nov 7, 2021
333d58b
spelling
abhinavsingh Nov 7, 2021
2af58e5
self.upstream = None only for pool config
abhinavsingh Nov 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
self.upstream = None only for pool config
  • Loading branch information
@abhinavsingh
abhinavsingh committed Nov 7, 2021
commit 2af58e5093366910bcad3ca9506c2affebcf5d2d
2 changes: 1 addition & 1 deletion proxy/http/proxy/server.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ def _close_and_release(self) -> bool:
self.upstream.closed = True
with self.lock:
self.pool.release(self.upstream)
self.upstream = None
self.upstream = None
return True

def write_to_descriptors(self, w: Writables) -> bool:
Expand Down