NaoyaFukuma / webserv

This is when you finally understand why a URL starts with HTTP
3 stars 0 forks source link

SocketBuff::AddStringを修正 #32

Closed tomofuji-dev closed 1 year ago

NaoyaFukuma commented 1 year ago

現状、AddString時に、データ量に合わせてバッファとして使っているメモリの動的確保が起こっていることで、 Addstring時にデータの欠損が起きている可能性がある。

エコーサーバー時点では、このまま開発をすすめるが、 HTTPリクエストのパースを実装したあとも同様の問題を抱えるなら、 バッファを固定長にすることで解決を試みることを検討したいと思う。

NaoyaFukuma commented 1 year ago

vectorによるバッファリングの実装に変更し回収した