Skip to content

Add new onRequestSent callback method #434

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
Closed
slandelle opened this issue Dec 12, 2013 · 2 comments
Closed

Add new onRequestSent callback method #434

slandelle opened this issue Dec 12, 2013 · 2 comments
Assignees
@slandelle
Milestone
1.7.23

Comments

@slandelle
Copy link
Contributor

Notify the callback when a request is being written on the wire.

If the original request causes multiple requests to be sent, for example, because of authorization or retry, it will be notified multiple times.
@ghost ghost assigned slandelle Dec 12, 2013
@slandelle
Copy link
Contributor Author

Implemented for Netty provider in AHC 1.7.x branch in 1fa45b4

@slandelle
Copy link
Contributor Author

@rlubke WDYT? Willing to support this on the Grizzly provider?

@slandelle slandelle mentioned this issue Dec 12, 2013
Closed
rlubke added a commit that referenced this issue Dec 12, 2013
@rlubke
Changes for #434.
b2d2434
@slandelle slandelle mentioned this issue Dec 18, 2013
Closed
cs-workco pushed a commit to cs-workco/async-http-client that referenced this issue Apr 13, 2023
@fabianfett
[HTTPConnectionPool] Fix timer action races (AsyncHttpClient#434)
0bce647 
- `_idleTimer` and `_backoffTimer` are protected by `stateLock`
- Added a new `struct Actions` that splits up actions from the state machine into actions that need to be executed inside the `stateLock` and outside in `stateLock`
- Add HTTP/1.1 connection pool stress test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slandelle slandelle

Labels
None yet
Projects
None yet
Milestone
1.7.23
Development

No branches or pull requests
1 participant
@slandelle