Just use WebSocketLogger for now. I don't think we really need to move this to its own module. 

I would like to come up with a better package name though than highlevel. 

Norman Maurer wrote:
> In 
> websockets/src/main/java/io/undertow/websockets/highlevel/impl/DefaultBinaryFrameSender.java:
>
> >  +                    written +=w;
> >  +                }
> >  +            }
> >  +            StreamSinkUtils.shutdownAndFlush(sink, callback);
> >  +
> >  +        } catch (IOException e) {
> >  +            callback.onError(e);
> >  +        }
> >  +    }
> >  +
> >  +    @Override
> >  +    public void sendBinary(final FileChannel payloadChannel, final int offset, final long length, final SendCallback callback) {
> >  +        try{
> >  +            if (length>  payloadChannel.size() - offset) {
> >  +                // TODO: Make use of localization
> >  +                throw new IllegalArgumentException("specified length is bigger the availble size of the FileChannel");
>
> I agree.. do we want to split it in an extra module first ? Or just 
> use the WebSocketLogger ?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/undertow-io/undertow/pull/33/files#r2839195>.
>