Skip to content

Commit 51653d8

Browse files
committed
Suffix extensions with Attempt
1 parent c6500ad commit 51653d8

File tree

9 files changed

+15
-17
lines changed

9 files changed

+15
-17
lines changed

client/src/main/java/org/asynchttpclient/Realm.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ public String getPassword() {
101101
}
102102

103103
public AuthScheme getScheme() {
104-
105104
return scheme;
106105
}
107106

client/src/main/java/org/asynchttpclient/handler/AsyncHandlerExtensions.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ public interface AsyncHandlerExtensions {
6060
*
6161
* @param remoteAddress the address we try to connect to
6262
*/
63-
void onTcpConnect(InetSocketAddress remoteAddress);
63+
void onTcpConnectAttempt(InetSocketAddress remoteAddress);
6464

6565
/**
6666
* Notify the callback after a successful connect
@@ -85,7 +85,7 @@ public interface AsyncHandlerExtensions {
8585
/**
8686
* Notify the callback before TLS handshake
8787
*/
88-
void onTlsHandshake();
88+
void onTlsHandshakeAttempt();
8989

9090
/**
9191
* Notify the callback after the TLS was successful
@@ -104,7 +104,7 @@ public interface AsyncHandlerExtensions {
104104
/**
105105
* Notify the callback when trying to fetch a connection from the pool.
106106
*/
107-
void onConnectionPool();
107+
void onConnectionPoolAttempt();
108108

109109
/**
110110
* Notify the callback when a new connection was successfully fetched from the pool.

client/src/main/java/org/asynchttpclient/handler/ExtendedAsyncHandler.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public void onHostnameResolutionFailure(String name, Throwable cause) {
3636
}
3737

3838
@Override
39-
public void onTcpConnect(InetSocketAddress address) {
39+
public void onTcpConnectAttempt(InetSocketAddress address) {
4040
}
4141

4242
@Override
@@ -48,7 +48,7 @@ public void onTcpConnectFailure(InetSocketAddress remoteAddress, Throwable cause
4848
}
4949

5050
@Override
51-
public void onTlsHandshake() {
51+
public void onTlsHandshakeAttempt() {
5252
}
5353

5454
@Override
@@ -60,7 +60,7 @@ public void onTlsHandshakeFailure(Throwable cause) {
6060
}
6161

6262
@Override
63-
public void onConnectionPool() {
63+
public void onConnectionPoolAttempt() {
6464
}
6565

6666
@Override

client/src/main/java/org/asynchttpclient/netty/channel/NettyConnectListener.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public void onSuccess(Channel channel) throws Exception {
9090
final AsyncHandlerExtensions asyncHandlerExtensions = toAsyncHandlerExtensions(future.getAsyncHandler());
9191

9292
if (asyncHandlerExtensions != null)
93-
asyncHandlerExtensions.onTlsHandshake();
93+
asyncHandlerExtensions.onTlsHandshakeAttempt();
9494

9595
sslHandler.handshakeFuture().addListener(new SimpleGenericFutureListener<Channel>() {
9696

client/src/main/java/org/asynchttpclient/netty/request/NettyChannelConnector.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public void connect(final Bootstrap bootstrap, final NettyConnectListener<?> con
4848
final InetSocketAddress remoteAddress = remoteAddresses.get(i);
4949

5050
if (asyncHandlerExtensions != null)
51-
asyncHandlerExtensions.onTcpConnect(remoteAddress);
51+
asyncHandlerExtensions.onTcpConnectAttempt(remoteAddress);
5252

5353
final ChannelFuture future = localAddress != null ? bootstrap.connect(remoteAddress, localAddress) : bootstrap.connect(remoteAddress);
5454

client/src/main/java/org/asynchttpclient/netty/request/NettyRequestSender.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -470,7 +470,7 @@ else if (!request.getMethod().equals(GET))
470470
private Channel pollPooledChannel(Request request, ProxyServer proxy, AsyncHandler<?> asyncHandler) {
471471

472472
if (asyncHandler instanceof AsyncHandlerExtensions)
473-
AsyncHandlerExtensions.class.cast(asyncHandler).onConnectionPool();
473+
AsyncHandlerExtensions.class.cast(asyncHandler).onConnectionPoolAttempt();
474474

475475
Uri uri = request.getUri();
476476
String virtualHost = request.getVirtualHost();

client/src/main/java/org/asynchttpclient/netty/ssl/JsseSslEngineFactory.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,5 +32,4 @@ public SSLEngine newSslEngine(AsyncHttpClientConfig config, String peerHost, int
3232
configureSslEngine(sslEngine, config);
3333
return sslEngine;
3434
}
35-
3635
}

client/src/test/java/org/asynchttpclient/netty/handler/NettyReactiveStreamsTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -133,19 +133,19 @@ public void onHostnameResolutionSuccess(String name, List<InetSocketAddress> add
133133
@Override
134134
public void onHostnameResolutionFailure(String name, Throwable cause) {}
135135
@Override
136-
public void onTcpConnect(InetSocketAddress address) {}
136+
public void onTcpConnectAttempt(InetSocketAddress address) {}
137137
@Override
138138
public void onTcpConnectSuccess(InetSocketAddress address, Channel connection) {}
139139
@Override
140140
public void onTcpConnectFailure(InetSocketAddress address, Throwable cause) {}
141141
@Override
142-
public void onTlsHandshake() {}
142+
public void onTlsHandshakeAttempt() {}
143143
@Override
144144
public void onTlsHandshakeSuccess() {}
145145
@Override
146146
public void onTlsHandshakeFailure(Throwable cause) {}
147147
@Override
148-
public void onConnectionPool() {}
148+
public void onConnectionPoolAttempt() {}
149149
@Override
150150
public void onConnectionPooled(Channel connection) {}
151151
@Override

client/src/test/java/org/asynchttpclient/test/EventCollectingHandler.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ public State onContentWritten() {
9595
}
9696

9797
@Override
98-
public void onTcpConnect(InetSocketAddress address) {
98+
public void onTcpConnectAttempt(InetSocketAddress address) {
9999
firedEvents.add(CONNECTION_OPEN_EVENT);
100100
}
101101

@@ -125,7 +125,7 @@ public void onHostnameResolutionFailure(String name, Throwable cause) {
125125
}
126126

127127
@Override
128-
public void onTlsHandshake() {
128+
public void onTlsHandshakeAttempt() {
129129
firedEvents.add(TLS_HANDSHAKE_EVENT);
130130
}
131131

@@ -140,7 +140,7 @@ public void onTlsHandshakeFailure(Throwable cause) {
140140
}
141141

142142
@Override
143-
public void onConnectionPool() {
143+
public void onConnectionPoolAttempt() {
144144
firedEvents.add(CONNECTION_POOL_EVENT);
145145
}
146146

0 commit comments

Comments
 (0)