From bfabd1fe57dae9e00f1f5b703ed72f851aa81e99 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Joakim=20Frosteg=C3=A5rd?= Date: Fri, 3 Jul 2020 14:04:37 +0200 Subject: [PATCH] Update TODO --- TODO.md | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/TODO.md b/TODO.md index 87771c3..c272b30 100644 --- a/TODO.md +++ b/TODO.md @@ -10,9 +10,16 @@ * test tls * request parsing in protocol module instead of in network? Not obvious what error return type to use then -* compact peer representation in announce response: is implementation correct? -* scrape info hash parsing: multiple ought to be accepted -* scrape response panic +* scrape info hash parsing: multiple ought to be accepted, might need to roll + my own url encode crate anyway or possibly use https://github.com/samscott89/serde_qs +* serialization + * there is the question of how serialization should be done for 20 byte + arrays, such as in the scrape response. There, a 20 byte byte string is + expected. I think the bendy AsString wrapper is what is needed here, but + the question is how to combine it with serde serialization AND how to + combine that with future serialization for load tester, where no bencode + should be involved. + * compact peer representation in announce response: is implementation correct? * move stuff to common crate with ws: what about Request/InMessage etc? * don't overdo this * 20 byte helper