I think we should aggregate here by ourself as we only need for upgrade requests

> Am 13.05.2014 um 21:28 schrieb Nathan Mittler <notifications@github.com>:
> 
> In codec-http2/src/main/java/io/netty/handler/codec/http2/HttpProtocolUpgradeHandler.java:
> 
> > +            throw new NullPointerException("initialProtocol");
> > +        }
> > +        if (supportedProtocols == null) {
> > +            throw new NullPointerException("supportedProtocols");
> > +        }
> > +        this.initialProtocol = initialProtocol;
> > +        protocolMap = new LinkedHashMap<String, TargetProtocol>(supportedProtocols.size());
> > +        for (TargetProtocol protocol : supportedProtocols) {
> > +            String name = protocol.name().toUpperCase();
> > +            protocolMap.put(name, protocol);
> > +        }
> > +    }
> > +
> > +    @Override
> > +    public void channelRead(ChannelHandlerContext ctx, Object msg) throws Exception {
> > +        if (msg instanceof HttpRequest) {
> Would it be sufficient to assume the aggregation has already been done and just look for instanceof FullHttpRequest?
> 
> —
> Reply to this email directly or view it on GitHub.