Skip to content

Commit 07e728b

Browse files
committed
refactoring
1 parent 3aed1de commit 07e728b

2 files changed

Lines changed: 16 additions & 15 deletions

File tree

rlib-network/src/main/java/com/ss/rlib/network/packet/impl/AbstractSSLPacketReader.java

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
import javax.net.ssl.SSLEngineResult;
1717
import javax.net.ssl.SSLEngineResult.HandshakeStatus;
1818
import javax.net.ssl.SSLException;
19+
import java.io.IOException;
1920
import java.nio.ByteBuffer;
2021
import java.nio.channels.AsynchronousSocketChannel;
2122

@@ -32,7 +33,6 @@ public abstract class AbstractSSLPacketReader<R extends ReadablePacket, C extend
3233
NetworkUtils.EMPTY_BUFFER
3334
};
3435

35-
private static final int MINIMAL_SSL_BUFFER = 512;
3636
private static final int SKIP_READ_PACKETS = -1;
3737

3838
protected final @NotNull SSLEngine sslEngine;
@@ -63,11 +63,6 @@ protected AbstractSSLPacketReader(
6363
return sslNetworkBuffer;
6464
}
6565

66-
@Override
67-
public void startRead() {
68-
super.startRead();
69-
}
70-
7166
@Override
7267
protected void handleReadData(@NotNull Integer result) {
7368

@@ -241,12 +236,11 @@ private void increaseDataBuffer() {
241236
}
242237

243238
protected void closeConnection() {
244-
/* try {
239+
try {
245240
sslEngine.closeOutbound();
246-
//doHandshake(readBuffer.clear(), 0);
247241
channel.close();
248242
} catch (IOException e) {
249243
throw new RuntimeException(e);
250-
}*/
244+
}
251245
}
252246
}

rlib-network/src/main/java/com/ss/rlib/network/packet/impl/AbstractSSLPacketWriter.java

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import javax.net.ssl.SSLEngineResult;
1919
import javax.net.ssl.SSLEngineResult.HandshakeStatus;
2020
import javax.net.ssl.SSLException;
21+
import java.io.IOException;
2122
import java.nio.ByteBuffer;
2223
import java.nio.channels.AsynchronousSocketChannel;
2324

@@ -76,11 +77,10 @@ public AbstractSSLPacketWriter(
7677

7778
switch (result.getStatus()) {
7879
case BUFFER_UNDERFLOW:
79-
// peerNetData = handleBufferUnderflow(engine, peerNetData);
80+
increaseNetworkBuffer();
8081
break;
8182
case BUFFER_OVERFLOW:
82-
// peerAppData = enlargeApplicationBuffer(engine, peerAppData);
83-
break;
83+
throw new IllegalStateException("Unexpected state");
8484
case OK:
8585
return sslNetworkBuffer.flip();
8686
case CLOSED:
@@ -187,13 +187,20 @@ public AbstractSSLPacketWriter(
187187
return null;
188188
}
189189

190+
private void increaseNetworkBuffer() {
191+
sslNetworkBuffer = NetworkUtils.increasePacketBuffer(
192+
sslNetworkBuffer,
193+
bufferAllocator,
194+
sslEngine
195+
);
196+
}
197+
190198
protected void closeConnection() {
191-
/**try {
199+
try {
192200
sslEngine.closeOutbound();
193-
doHandshake(NULL_PACKET);
194201
channel.close();
195202
} catch (IOException e) {
196203
throw new RuntimeException(e);
197-
}*/
204+
}
198205
}
199206
}

0 commit comments

Comments
 (0)