Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

HTTP/2 Use of ByteBuf for frames with small content

See original GitHub issue

Currently Http2PingFrame and Http2GoAwayFrame are backed by a ByteBuf. This was done for consistency reasons but increases the complexity for sharing/reusing these objects. Since is it assumed the payload size for these frames is typically small (<100 bytes) should we instead just use a byte[] for these frames to simplify the APIs?

Issue Analytics

State:
Created 6 years ago
Comments:21 (21 by maintainers)

Top GitHub Comments

2reactions

Scottmitchcommented, Feb 1, 2018

+1 for changing it all lets take advantage of UnstableApi while we still can 😃

0reactions

normanmaurercommented, Feb 8, 2018

Fixed by https://github.com/netty/netty/pull/7690

Top Results From Across the Web

Uses of Class io.netty.buffer.ByteBuf

Creates a new big-endian Java heap buffer with reasonably small initial capacity, which expands its capacity boundlessly on demand.

Http2MultiplexCodec leaks ping payloads · Issue #7607 · netty ...

Start HTTP/2 server and send it ping frames with advanced leak detection ... ByteBuf.release() was not called before it's garbage-collected.

HTTP/2 in Netty | Baeldung

Communication in HTTP/2 happens via a group of bytes called frames, and multiple frames form a stream. In our code samples, we'll see...

io.netty.handler.codec.http2.Http2ConnectionEncoder java code ...

How to use ... Best Java code snippets using io.netty.handler.codec.http2. ... void onDataAvailable(final ByteString data) { ByteBuf content = Unpooled.

RSocket for the internet: prototyping Http/2 based transport

Lets start with RSocket 0 stream frames. Keep-alives are translated to Http2 PING frames with small data payload of 8 bytes - just...