From 5896cd51de65cb6f7b8f3a4df85c4eb8886b76e2 Mon Sep 17 00:00:00 2001
From: Aleksey Kladov <aleksey.kladov@gmail.com>
Date: Fri, 10 Aug 2018 22:55:42 +0300
Subject: Cleanup log messages

---
 crates/server/src/main.rs | 45 ++++++++++++++++++++++++++++++++++-----------
 1 file changed, 34 insertions(+), 11 deletions(-)

(limited to 'crates/server/src')

diff --git a/crates/server/src/main.rs b/crates/server/src/main.rs
index 116abce1c..2d8695d23 100644
--- a/crates/server/src/main.rs
+++ b/crates/server/src/main.rs
@@ -41,10 +41,10 @@ fn main() -> Result<()> {
         .log_to_file()
         .directory("log")
         .start()?;
-    info!("starting server");
+    info!("lifecycle: server started");
     match ::std::panic::catch_unwind(|| main_inner()) {
         Ok(res) => {
-            info!("shutting down: {:?}", res);
+            info!("lifecycle: terminating process with {:?}", res);
             res
         }
         Err(_) => {
@@ -105,15 +105,31 @@ fn initialize(io: &mut Io) -> Result<()> {
 type Thunk = Box<for<'a> FnBox<&'a mut Io, Result<()>>>;
 
 fn initialized(io: &mut Io) -> Result<()> {
-    let mut world = WorldState::new();
-    let mut pool = ThreadPool::new(4);
-    let (sender, receiver) = bounded::<Thunk>(16);
-    let res = main_loop(io, &mut world, &mut pool, sender, receiver.clone());
-    info!("waiting for background jobs to finish...");
-    receiver.for_each(drop);
-    pool.join();
-    info!("...background jobs have finished");
-    res
+    {
+        let mut world = WorldState::new();
+        let mut pool = ThreadPool::new(4);
+        let (sender, receiver) = bounded::<Thunk>(16);
+        info!("lifecycle: handshake finished, server ready to serve requests");
+        let res = main_loop(io, &mut world, &mut pool, sender, receiver.clone());
+        info!("waiting for background jobs to finish...");
+        receiver.for_each(drop);
+        pool.join();
+        info!("...background jobs have finished");
+        res
+    }?;
+
+    match io.recv()? {
+        RawMsg::Notification(n) => {
+            if n.method == "exit" {
+                info!("lifecycle: shutdown complete");
+                return Ok(());
+            }
+            bail!("unexpected notification during shutdown: {:?}", n)
+        }
+        m => {
+            bail!("unexpected message during shutdown: {:?}", m)
+        }
+    }
 }
 
 fn main_loop(
@@ -172,10 +188,17 @@ fn main_loop(
                     });
                     Ok(())
                 })?;
+                let mut shutdown = false;
                 dispatch::handle_request::<req::Shutdown, _>(&mut req, |(), resp| {
                     resp.result(io, ())?;
+                    shutdown = true;
                     Ok(())
                 })?;
+                if shutdown {
+                    info!("lifecycle: initiating shutdown");
+                    drop(sender);
+                    return Ok(());
+                }
                 if let Some(req) = req {
                     error!("unknown method: {:?}", req);
                     dispatch::unknown_method(io, req)?;
-- 
cgit v1.2.3