-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Error Logging in FastHttpUser #2937
Comments
PR:s welcome (I'll look at it myself, but no guarantees on the timing) |
I would like to work on this issue. Please assign me this issue. |
Ideally, make it work the same as HttpUser. |
I made this change to make the URL parameter optional by modifying the repository for geventhttpuser, then modified the use of that class in the Locust repository by removing the use of the url parameter in the "LocustUserAgent" class in fasthttp.py. geventhttpclient:swaalt/geventhttpclient@94253d4 fasthttp:swaalt@ef4d4c3 |
Interesting. But maybe even better/easier to subclass the ConnectionError class (in locust) and override the |
Would that be okay? |
Looks great! PR it? Maybe add a test case for it? (ignore any other errors, the tests are a bit flaky atm) |
Attach the PR : #2944 |
Discussed in https://github.com/orgs/locustio/discussions/2936
Originally posted by swaalt October 10, 2024
Hi, I am using FastHttpUser and when I receive a 404 the fail is registered like this, this happens because the geventhttpclient saves in the exception the badstatuscode text the url of the request I made, which is dynamic and appears separated by each one,
I need that either I take the “name” or simply do not place anything, just place the badstatuscode 404, can this be addressed with the locust as it is now or do I have to modify something?
The text was updated successfully, but these errors were encountered: