From 4e3a42d77d23ac59b58b6e0d17efd12c61a06c9d Mon Sep 17 00:00:00 2001
From: dowwie <dkcdkg@gmail.com>
Date: Sat, 1 Jun 2019 09:24:54 -0400
Subject: [PATCH 1/2] changed order of logging middleware, which must register
 last

---
 basics/src/main.rs        | 4 ++--
 cookie-auth/src/main.rs   | 2 +-
 redis-session/src/main.rs | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/basics/src/main.rs b/basics/src/main.rs
index 5d74e3d9..b7406f99 100644
--- a/basics/src/main.rs
+++ b/basics/src/main.rs
@@ -82,10 +82,10 @@ fn main() -> io::Result<()> {
 
     HttpServer::new(|| {
         App::new()
-            // enable logger
-            .wrap(middleware::Logger::default())
             // cookie session middleware
             .wrap(CookieSession::signed(&[0; 32]).secure(false))
+            // enable logger
+            .wrap(middleware::Logger::default())
             // register favicon
             .service(favicon)
             // register simple route, handle all methods
diff --git a/cookie-auth/src/main.rs b/cookie-auth/src/main.rs
index 9d1f8577..2e428014 100644
--- a/cookie-auth/src/main.rs
+++ b/cookie-auth/src/main.rs
@@ -22,12 +22,12 @@ fn main() -> std::io::Result<()> {
 
     HttpServer::new(|| {
         App::new()
-            .wrap(middleware::Logger::default())
             .wrap(IdentityService::new(
                 CookieIdentityPolicy::new(&[0; 32])
                     .name("auth-example")
                     .secure(false),
             ))
+            .wrap(middleware::Logger::default())
             .service(web::resource("/login").route(web::post().to(login)))
             .service(web::resource("/logout").to(logout))
             .service(web::resource("/").route(web::get().to(index)))
diff --git a/redis-session/src/main.rs b/redis-session/src/main.rs
index 0c49c6bf..2705866f 100644
--- a/redis-session/src/main.rs
+++ b/redis-session/src/main.rs
@@ -26,10 +26,10 @@ fn main() -> std::io::Result<()> {
 
     HttpServer::new(|| {
         App::new()
-            // enable logger
-            .wrap(middleware::Logger::default())
             // redis session middleware
             .wrap(RedisSession::new("127.0.0.1:6379", &[0; 32]))
+            // enable logger
+            .wrap(middleware::Logger::default())
             // register simple route, handle all methods
             .service(web::resource("/").to(index))
     })

From a8b49dfaec0c2e28fb486d50164ecdece93cc1ac Mon Sep 17 00:00:00 2001
From: dowwie <dkcdkg@gmail.com>
Date: Sat, 1 Jun 2019 09:27:34 -0400
Subject: [PATCH 2/2] updated code comment about log middleware

---
 basics/src/main.rs        | 2 +-
 cookie-auth/src/main.rs   | 1 +
 redis-session/src/main.rs | 2 +-
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/basics/src/main.rs b/basics/src/main.rs
index b7406f99..7a4edcb1 100644
--- a/basics/src/main.rs
+++ b/basics/src/main.rs
@@ -84,7 +84,7 @@ fn main() -> io::Result<()> {
         App::new()
             // cookie session middleware
             .wrap(CookieSession::signed(&[0; 32]).secure(false))
-            // enable logger
+            // enable logger - always register actix-web Logger middleware last 
             .wrap(middleware::Logger::default())
             // register favicon
             .service(favicon)
diff --git a/cookie-auth/src/main.rs b/cookie-auth/src/main.rs
index 2e428014..f8d9fc0b 100644
--- a/cookie-auth/src/main.rs
+++ b/cookie-auth/src/main.rs
@@ -27,6 +27,7 @@ fn main() -> std::io::Result<()> {
                     .name("auth-example")
                     .secure(false),
             ))
+            // enable logger - always register actix-web Logger middleware last 
             .wrap(middleware::Logger::default())
             .service(web::resource("/login").route(web::post().to(login)))
             .service(web::resource("/logout").to(logout))
diff --git a/redis-session/src/main.rs b/redis-session/src/main.rs
index 2705866f..ba840b42 100644
--- a/redis-session/src/main.rs
+++ b/redis-session/src/main.rs
@@ -28,7 +28,7 @@ fn main() -> std::io::Result<()> {
         App::new()
             // redis session middleware
             .wrap(RedisSession::new("127.0.0.1:6379", &[0; 32]))
-            // enable logger
+            // enable logger - always register actix-web Logger middleware last 
             .wrap(middleware::Logger::default())
             // register simple route, handle all methods
             .service(web::resource("/").to(index))