nopara73 / DotNetTor

Library implementation of essential Tor features in .NET Core.
https://www.nuget.org/packages/DotNetTor
MIT License
77 stars 21 forks source link

setting the length to be the size of the bytes not the length of the string #12

Closed mikeapple closed 6 years ago

mikeapple commented 6 years ago

This makes sense to me now (ignore my last PR). The WriteAsync was being set of 0 to length of the string which works for ascii which is 1 byte per char but as utf-8 is 2 then the byte size is greater than the length of the string.

nopara73 commented 6 years ago

Good catch!