Potential place for 499 integration

Signed-off-by: Eloi DEMOLIS <eloi.demolis@clever-cloud.com>
sozu-proxy · Sep 7, 2022 · 7b30066 · 7b30066
1 parent 5e77738
commit 7b30066
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/src/protocol/http/mod.rs b/lib/src/protocol/http/mod.rs
@@ -1800,6 +1800,10 @@ impl<Front: SocketHandler, L: ListenerHandler> Http<Front, L> {
                     self.res_header_end = header_end2;
                 };
 
+                // we may check for 499 with get_status_line
+                // here and return (ProtocolResult::Continue, SessionResult::CloseSession)
+                // this should close the connection without sending a response to the client
+
                 if unwrap_msg!(self.response_state.as_ref()).is_back_error() {
                     self.set_answer(DefaultAnswerStatus::Answer502, None);
                     return (ProtocolResult::Continue, self.writable(metrics));

diff --git a/lib/src/protocol/http/parser/response.rs b/lib/src/protocol/http/parser/response.rs
@@ -269,10 +269,6 @@ pub fn parse_response(
             match status_line(buf) {
                 Ok((input, raw_status_line)) => {
                     match StatusLine::from_raw_status_line(raw_status_line) {
-                        // We may handle 499
-                        // Some(499) => {
-                        //     (BufferMove::None, ResponseState::Error())
-                        // }
                         Some(status_line) => {
                             let conn = Connection::new();
                             /*let conn = if rl.version == "11" {