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 proxy support for FastHttpUser #2758

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Conversation

NicoAdrian
Copy link

No description provided.

@cyberw
Copy link
Collaborator

cyberw commented Jun 10, 2024

Nice! While I think you could just provide your own clientpool to achieve the same thing, we're already allowing some parameters for convenience so I guess this is fine.

I wont merge just yet, because I want to make a "clean" release with only the Poetry change in another open PR, but it should be soon (this week hopefully)

@NicoAdrian
Copy link
Author

Nice! While I think you could just provide your own clientpool to achieve the same thing, we're already allowing some parameters for convenience so I guess this is fine.

I wont merge just yet, because I want to make a "clean" release with only the Poetry change in another open PR, but it should be soon (this week hopefully)

With HTTPClientPool from geventhttpclient ? The thing is, I want all my users to use a different connection. Or I could just maybe instantiate it in my user's __init__ ?

@cyberw
Copy link
Collaborator

cyberw commented Jun 10, 2024

Or I could just maybe instantiate it in my user's __init__ ?

Yes, that what I meant :) But I don't expect people to do that, so this PR is fine.

@cyberw cyberw merged commit d4df346 into locustio:master Jun 25, 2024
14 checks passed
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