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

Add connection pool? #122

Open
daewok opened this issue Apr 19, 2022 · 1 comment
Open

Add connection pool? #122

daewok opened this issue Apr 19, 2022 · 1 comment

Comments

@daewok
Copy link

daewok commented Apr 19, 2022

Would there be interest in adding connection pool support to Drakma?

I had plans to create an external connection pool that wrapped http-request and passed it a :stream. But I realized this is ~impossible to get right when using a proxy, as Drakma will only send the full URI if :stream is nil. Plus there are some other fiddly bits that are hard to get right externally, like dealing with redirects.

I figured I'd see if there was interest in adding a connection pool directly to Drakma instead of continuing to bang my head against the wall to get an external pool to handle all the corner cases correctly.

@ajberkley
Copy link

Feel free to steal the connection-cache from dexador...
https://github.com/fukamachi/dexador/blob/master/src/connection-cache.lisp

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

No branches or pull requests

2 participants