actix: Document the situation with HEAD etc.

This is not very good really, but I guess it will do for now.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/TODO b/TODO
index 343b87d2567c539f47b66471f84969d5c1d4583c..a817962ad46f1a481c88d719028c7adabfb95ced 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,2 +1 @@
-http2 HEAD requests
 Build on stable

diff --git a/daemon/main.rs b/daemon/main.rs
index 0e105bd5fc2fb2335b12e1aa81553bc3ebf0cfdb..6b903724958d7edb2f156a5fceb95654c59b8bb5 100644 (file)
--- a/daemon/main.rs
+++ b/daemon/main.rs
@@ -161,6 +161,16 @@ struct LoadingRenderContext<'r> {
   movehist_len_i: usize,
   movehist_len_max: usize,
 }
+
+// This business with method="GET", method="HEAD" is necessary because
+// actix_web 4's #[get] macro does not serve HEAD requests.
+// Apparently, this is because they are broken in its HTTP/2
+// implementation.  But, also, the HTTP/2 implementation is not
+// exposed unless you have actix do TLS (which is where the version
+// negotiation ends up).  So this is fine?
+//
+// Upstream issue:
+//  https://github.com/actix/actix-web/issues/2702
 #[route("/", method="GET", method="HEAD")]
 #[throws(FER)]
 async fn r_loading_p(ia: PlayerQueryString,