HTTP Web Proxy using Non Blocking IO

Question

Resolved Thanks................

Answer 1

client.register(selector, SelectionKey.OP_READ
| SelectionKey.OP_WRITE);OP_READ only at this stage. OP_WRITE is always 'ready' unless the socket send buffer is full, so your selector will just spin. Don't register OP_WRITE until you have something to write.
ByteBuffer bferClient = ByteBuffer.allocate(4096);You're doing this every time around the loop, and you don't even know whether the key is readable yet. Do it when you accept a connection, and save it as the key attachment for that channel.
client.configureBlocking(false);It's already in non-blocking mode. Why are you doing this again?
try {
if ((bytesRead = client.read(bferClient)) != -1) {
bferClient.flip();
String msgFromClient = new String(bferClient.array());new String(bferClient.array(), 0, bferClient.limit()-1), but you're assuming you got a complete command here. You may not have. That's another reason why you need a ByteBuffer per channel.
channel.connect(socketAddress);You should do that in non-blocking mode, register the channel for OP_CONNECT, and let the select loop take care of finishing the connection.
channel.write(bferClient);The incoming command was CONNECT. You don't write that upstream.
if (!"".equals(msgFromClient.trim())) {
int bytesReadServer;
byte[] reply = new byte[4096];
try {
ByteBuffer bufferNIO = ByteBuffer.allocateDirect(1024);
while ((channel.read(bufferNIO)) != -1) {
bufferNIO.flip();
client.write(bufferNIO);
bufferNIO.clear();
}And you don't do any of this here. You register the new channel for OP_CONNECT, then when you get that do finishConnect() and then register it for OP_READ, and proceed around your select loop. You also need to associate this channel with the channel you read the CONNECT from. So you really need a key attachment object for the original channel that contains its byte buffer and the upstream channel; similarly the upstream channel needs a key attachment that contains its buffer and its downstream channel. The effect of doing all this stuff here instead of around the select loop is that you are never getting back to the select loop: you are handling this one client here, in blocking mode, until the connection is dropped. Completely wrong.
} catch (IOException e) {Hang on. If bytesRead was -1 you must close the channel.
e.printStackTrace();And if you get any IOException here you must also close the channel.
if (channel != null) {
try {
channel.close();This is a really strange place to close this channel.
When you get all that fixed up, post your code and I'll tell you how to handle OP_WRITE.