From af340f95de6c226abb36a86b9d774c4aeae55e6c Mon Sep 17 00:00:00 2001 From: Dario Date: Wed, 15 Jul 2020 13:45:50 +0200 Subject: [PATCH] update logging to better nail down mysql errors --- src/cpp/SingletonManager/ConnectionManager.h | 4 ++-- src/cpp/model/table/EmailOptIn.cpp | 3 +++ src/cpp/model/table/ModelBase.cpp | 2 +- src/cpp/model/table/User.cpp | 8 ++++---- 4 files changed, 10 insertions(+), 7 deletions(-) diff --git a/src/cpp/SingletonManager/ConnectionManager.h b/src/cpp/SingletonManager/ConnectionManager.h index 332f4c2d4..d7eb65ec3 100644 --- a/src/cpp/SingletonManager/ConnectionManager.h +++ b/src/cpp/SingletonManager/ConnectionManager.h @@ -43,8 +43,8 @@ public: throw Poco::NotFoundException("Connection Type unknown", std::to_string(type)); } auto session = mSessionPools.getPool(mSessionPoolNames[type]).get(); - std::string dateTimeString = Poco::DateTimeFormatter::format(Poco::DateTime(), "%d.%m.%y %H:%M:%S"); - printf("[getConnection] %s impl: %p\n", dateTimeString.data(), session.impl()); + //std::string dateTimeString = Poco::DateTimeFormatter::format(Poco::DateTime(), "%d.%m.%y %H:%M:%S"); + //printf("[getConnection] %s impl: %p\n", dateTimeString.data(), session.impl()); return session; } diff --git a/src/cpp/model/table/EmailOptIn.cpp b/src/cpp/model/table/EmailOptIn.cpp index c7871bef7..55358215b 100644 --- a/src/cpp/model/table/EmailOptIn.cpp +++ b/src/cpp/model/table/EmailOptIn.cpp @@ -43,6 +43,9 @@ namespace model { Poco::Data::Statement insert(session); lock(); + assert(mUserId > 0); + assert(mEmailVerificationCode > 0); + insert << "INSERT INTO " << getTableName() << " (user_id, verification_code, email_opt_in_type_id, resend_count) VALUES(?,?,?,?)" , use(mUserId), use(mEmailVerificationCode), bind(mType), bind(mResendCount); diff --git a/src/cpp/model/table/ModelBase.cpp b/src/cpp/model/table/ModelBase.cpp index 107b5104c..5ba5a5529 100644 --- a/src/cpp/model/table/ModelBase.cpp +++ b/src/cpp/model/table/ModelBase.cpp @@ -66,7 +66,7 @@ namespace model { } catch (Poco::Exception& ex) { addError(new ParamError(getTableName(), "mysql error by insert", ex.displayText().data())); - addError(new ParamError(getTableName(), "data set: ", toString().data())); + addError(new ParamError(getTableName(), "data set: \n", toString().data())); } //printf("data valid: %s\n", toString().data()); return false; diff --git a/src/cpp/model/table/User.cpp b/src/cpp/model/table/User.cpp index 6a5c92728..3349eb1d9 100644 --- a/src/cpp/model/table/User.cpp +++ b/src/cpp/model/table/User.cpp @@ -179,8 +179,8 @@ namespace model { return update.execute(); } catch (Poco::Exception& ex) { - addError(new ParamError(getTableName(), "mysql error by insert", ex.displayText().data())); - addError(new ParamError(getTableName(), "data set: ", toString().data())); + addError(new ParamError(getTableName(), "[updatePrivkeyAndPasswordHash] mysql error by update", ex.displayText().data())); + addError(new ParamError(getTableName(), "data set: \n", toString().data())); } //printf("data valid: %s\n", toString().data()); return 0; @@ -206,8 +206,8 @@ namespace model { return update.execute(); } catch (Poco::Exception& ex) { - addError(new ParamError(getTableName(), "mysql error by insert", ex.displayText().data())); - addError(new ParamError(getTableName(), "data set: ", toString().data())); + addError(new ParamError(getTableName(), "[updatePubkeyAndPrivkey] mysql error by update", ex.displayText().data())); + addError(new ParamError(getTableName(), "data set: \n", toString().data())); } //printf("data valid: %s\n", toString().data()); return 0;