search

Azure / DotNetty

DotNetty project – a port of netty, event-driven asynchronous network application framework
Other
4.09k stars 977 forks source link

LengthFieldPrepender error message wrong #349

Closed Iamcaibi closed 6 years ago

Iamcaibi commented 6 years ago

LengthFieldPrepender line 121 , can not set to 3,the error message is wrong

yyjdelete commented 6 years ago

And the same thing for LengthFieldBasedFrameDecoder.cs.

https://github.com/Azure/DotNetty/blob/dev/src/DotNetty.Codecs/LengthFieldBasedFrameDecoder.cs#L419 https://github.com/Azure/DotNetty/blob/dev/src/DotNetty.Codecs/LengthFieldBasedFrameDecoder.cs#L398

Maybe can add support for Medium(3)?

I found another issue in LengthFieldBasedFrameDecoder and will do an PR latter.

yyjdelete commented 6 years ago

see also #252