fix(timeout): implements cancel on drop

Co-Authored-By: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
Signed-off-by: Florentin Dubois <florentin.dubois@clever-cloud.com>

lib/src/https.rs

@@ -208,7 +208,7 @@ impl HttpsSession {
 
     fn upgrade_expect(
         &mut self,
-        expect: ExpectProxyProtocol<MioTcpStream>,
+        mut expect: ExpectProxyProtocol<MioTcpStream>,
         ssl: ServerConnection,
     ) -> Option<HttpsStateMachine> {
         if let Some(ref addresses) = expect.addresses {
@@ -245,7 +245,10 @@ impl HttpsSession {
         }
 
         // currently, only happens in expect proxy protocol with AF_UNSPEC address
-        // error!("failed to upgrade from expect");
+        if !expect.container_frontend_timeout.cancel() {
+            error!("failed to cancel request timeout on expect upgrade phase for 'expect proxy protocol with AF_UNSPEC address'");
+        }
+
         None
     }
 

lib/src/timer.rs

@@ -180,6 +180,14 @@ impl TimeoutContainer {
     }
 }
 
+impl std::ops::Drop for TimeoutContainer {
+    fn drop(&mut self) {
+        if self.cancel() {
+            error!("Cancel a dangling timeout that haven't be handled in session lifecycle, token ({:?}), duration {}", self.token, self.duration);
+        }
+    }
+}
+
 #[derive(Copy, Clone, Debug)]
 struct WheelEntry {
     next_tick: Tick,