From 57b2c763232f564d6076c7aaa04bda325829717a Mon Sep 17 00:00:00 2001
From: Fabian Posch <fabian@posch.tech>
Date: Mon, 8 Jan 2024 14:11:09 -0500
Subject: [PATCH] fixed throwing exception references instead of references

---
 src/cli_util.cpp | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/cli_util.cpp b/src/cli_util.cpp
index ea8fe91..6acf542 100644
--- a/src/cli_util.cpp
+++ b/src/cli_util.cpp
@@ -33,7 +33,7 @@ void get_db_cred_env (db::db_credentials_t& db_cred) {
     // check if a server was given
     if (!db_cred.server_override) {
         if (std::getenv(SERVER_ADDR_ENV) == nullptr) {
-            throw new std::invalid_argument("No database server set in environment variable " SERVER_ADDR_ENV);
+            throw std::invalid_argument("No database server set in environment variable " SERVER_ADDR_ENV);
         }
 
         db_cred.server = std::getenv(SERVER_ADDR_ENV);
@@ -58,12 +58,12 @@ void get_db_cred_env (db::db_credentials_t& db_cred) {
             try {
                 db_cred.port = std::atoi(port_st.c_str());
             } catch (std::invalid_argument& e) {
-                throw new std::invalid_argument("Could not parse port from environment variable " SERVER_PORT_ENV);
+                throw std::invalid_argument("Could not parse port from environment variable " SERVER_PORT_ENV);
             }
 
             // make sure the port is valid
             if (db_cred.port <= 0) {
-                throw new std::invalid_argument("Port " SERVER_PORT_ENV " given in environment variable is not a valid port.");
+                throw std::invalid_argument("Port " SERVER_PORT_ENV " given in environment variable is not a valid port.");
             }
 
             db_cred.port_override = false;
@@ -73,7 +73,7 @@ void get_db_cred_env (db::db_credentials_t& db_cred) {
     // check if a user was given
     if (!db_cred.uname_override) {
         if (std::getenv(SERVER_USER_ENV) == nullptr) {
-            throw new std::invalid_argument("No database user set in environment variable " SERVER_USER_ENV);
+            throw std::invalid_argument("No database user set in environment variable " SERVER_USER_ENV);
         }
         db_cred.uname = std::getenv(SERVER_USER_ENV);
         db_cred.uname_override = false;
@@ -82,7 +82,7 @@ void get_db_cred_env (db::db_credentials_t& db_cred) {
     // check if a password was given
     if (!db_cred.pwd_override) {
         if (std::getenv(SERVER_PWD_ENV) == nullptr) {
-            throw new std::invalid_argument("No database password set in environment variable " SERVER_PWD_ENV);
+            throw std::invalid_argument("No database password set in environment variable " SERVER_PWD_ENV);
         }
         db_cred.pwd = std::getenv(SERVER_PWD_ENV);
         db_cred.pwd_override = false;
@@ -91,7 +91,7 @@ void get_db_cred_env (db::db_credentials_t& db_cred) {
     // check if a database was given
     if (!db_cred.dbase_override) {
         if (std::getenv(SERVER_DBASE_ENV) == nullptr) {
-            throw new std::invalid_argument("No database set in environment variable " SERVER_DBASE_ENV);
+            throw std::invalid_argument("No database set in environment variable " SERVER_DBASE_ENV);
         }
         db_cred.dbase = std::getenv(SERVER_DBASE_ENV);
         db_cred.dbase_override = false;