commit bd078c3b730914fb4edc46717d92106d49d6c4b8
author Patrick Williams <patrick@stwcx.xyz> Fri Sep 03 08:27:12 2021 -0500
committer Ed Tanous <ed@tanous.net> Fri Sep 03 19:15:18 2021 +0000
tree bccdc7f3f312bdd2dcf2c91e4d30a749f14a0e3f
parent a1978347588642a9677eca4d96d2a89e93d225a0

exception: catch by reference

`catch` should always be done by reference to avoid object slicing and
excess copy-constructor calls.  Preference is for exceptions to also
be caught 'const'.  (Both of these come from the C++ Core Guidelines)

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I7753abbef218a87ceecbd068245c14f413fadabc

src/security_headers_middleware_test.cpp

diff --git a/src/security_headers_middleware_test.cpp b/src/security_headers_middleware_test.cpp
index f5cd3d5..80df725 100644
--- a/src/security_headers_middleware_test.cpp
+++ b/src/security_headers_middleware_test.cpp
@@ -33,7 +33,7 @@
                 c.close();
                 break;
             }
-            catch (std::exception e)
+            catch (const std::exception& e)
             {
                 // do nothing.  We expect this to fail while the server is
                 // starting up