Can't make request chunking work with Netty 4.0.6

[slandelle]

Hi,

I'm desperately trying to make ChunkedStream work on the client side, ie for chuking a request body.

Here's what my code look like (a similar code works fine with Netty 3).

plainBootstrap.handler(new ChannelInitializer<Channel>() {

  protected void initChannel(Channel ch) throws Exception {
    ChannelPipeline pipeline = ch.pipeline();
    pipeline.addLast("httpHandler", new HttpClientCodec());
    pipeline.addLast("chunkedWriter", new ChunkedWriteHandler());
    pipeline.addLast("httpProcessor", new WriteStreamHandler(latch, status, responseBodySize));
  }
});

ChannelFuture f = plainBootstrap.connect(new InetSocketAddress("127.0.0.1", port1));

f.addListener(new ChannelFutureListener() {
  public void operationComplete(ChannelFuture future) throws Exception {
    DefaultFullHttpRequest request = new DefaultFullHttpRequest(HttpVersion.HTTP_1_1, HttpMethod.POST, getTargetUrl());
    request.headers().add(HttpHeaders.Names.ACCEPT, "*/*");
    request.headers().set(HttpHeaders.Names.TRANSFER_ENCODING, HttpHeaders.Values.CHUNKED);
    request.headers().add(HttpHeaders.Names.HOST, "*127.0.0.1:" + port1);
    request.headers().add(HttpHeaders.Names.CONNECTION, HttpHeaders.Values.KEEP_ALIVE);

    future.channel().write(request);
    future.channel().write(new ChunkedStream(new ByteArrayInputStream("hello".getBytes())));
    future.channel().writeAndFlush(LastHttpContent.EMPTY_LAST_CONTENT);
  }
});

I've created a test case here: https://github.com/slandelle/netty-request-chunking

Regards,

Stéphane

[normanmaurer]

@slandelle fixed by 60b8893 . Please retest.

[slandelle]

Will do, thanks!

2013/8/7 Norman Maurer notifications@github.com

@slandelle https://github.com/slandelle fixed by 60b889360b8893. Please retest.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1693#issuecomment-22235098
.

[normanmaurer]

I just notice you use FullHttpRequest… what you want to use is HttpRequest.

Am 07.08.2013 um 10:31 schrieb Stephane Landelle notifications@github.com:

Will do, thanks!

2013/8/7 Norman Maurer notifications@github.com

@slandelle https://github.com/slandelle fixed by 60b889360b8893. Please retest.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1693#issuecomment-22235098
.

—
Reply to this email directly or view it on GitHub.

[slandelle]

I've updated my sample (https://github.com/slandelle/netty-request-chunking) but it's still not working. Am I doing something wrong?

[normanmaurer]

@slandelle your test has a bug... pull in and re-run: https://github.com/slandelle/netty-request-chunking/pull/1