Introduce HandlerUserSettings to change user settings, e. g. pw changes
This commit is contained in:
parent
ac99894157
commit
1a79eaac51
@ -32,6 +32,7 @@ SOFTWARE.
|
||||
#include "handlercategory.h"
|
||||
#include "handlerhistory.h"
|
||||
#include "handlerpagedelete.h"
|
||||
#include "handlerusersettings.h"
|
||||
|
||||
std::unique_ptr<Handler> HandlerFactory::createHandler(const std::string &action, Session &userSession)
|
||||
{
|
||||
@ -75,6 +76,10 @@ std::unique_ptr<Handler> HandlerFactory::createHandler(const std::string &action
|
||||
{
|
||||
return produce<HandlerHistory>(userSession);
|
||||
}
|
||||
if(action == "usersettings")
|
||||
{
|
||||
return produce<HandlerUserSettings>(userSession);
|
||||
}
|
||||
|
||||
return produce<HandlerInvalidAction>(userSession);
|
||||
}
|
||||
|
71
handlers/handlerusersettings.cpp
Normal file
71
handlers/handlerusersettings.cpp
Normal file
@ -0,0 +1,71 @@
|
||||
#include "handlerusersettings.h"
|
||||
#include "../authenticator.h"
|
||||
#include "../random.h"
|
||||
|
||||
Response HandlerUserSettings::handleRequest(const Request &r)
|
||||
{
|
||||
if(r.getRequestMethod() == "POST")
|
||||
{
|
||||
if(r.post("do") == "submit")
|
||||
{
|
||||
std::string oldpassword = r.post("oldpassword");
|
||||
std::string newpassword = r.post("newpassword");
|
||||
std::string newpasswordconfirm = r.post("newpasswordconfirm");
|
||||
|
||||
if(newpassword != newpasswordconfirm)
|
||||
{
|
||||
//TODO: is not nice, users has to hit the back button...
|
||||
return this->errorResponse("Passwords don't match", "The entered new passwords don't match");
|
||||
}
|
||||
auto userDao = this->database->createUserDao();
|
||||
Authenticator authenticator(*userDao);
|
||||
|
||||
std::variant<User, AuthenticationError> authresult = authenticator.authenticate(this->userSession->user.login, oldpassword);
|
||||
if(std::holds_alternative<AuthenticationError>(authresult))
|
||||
{
|
||||
return this->errorResponse("Invalid current password", "The old password you entered is invalid");
|
||||
}
|
||||
Random r;
|
||||
std::vector<char> salt = r.getRandom(23);
|
||||
User user = std::get<User>(authresult);
|
||||
user.salt = salt;
|
||||
user.password = authenticator.hash(newpassword, user.salt);
|
||||
if(user.password.empty())
|
||||
{
|
||||
Logger::error() << "Authenticator pbkdf5 returned emptys hash";
|
||||
return this->errorResponse("Error", "An error occured while trying to store new password");
|
||||
}
|
||||
try
|
||||
{
|
||||
userDao->save(user);
|
||||
}
|
||||
catch(const DatabaseException &e)
|
||||
{
|
||||
Logger::debug() << "Error saving user: " << e.what();
|
||||
return errorResponse("Error", "A database error occured while trying to save user with new settings");
|
||||
}
|
||||
|
||||
return Response::redirectTemporarily(this->urlProvider->userSettings());
|
||||
}
|
||||
}
|
||||
|
||||
TemplatePage &userSettingsPage = this->templ->getPage("usersettings");
|
||||
setGeneralVars(userSettingsPage);
|
||||
userSettingsPage.setVar("usersettingsurl", urlProvider->userSettings());
|
||||
userSettingsPage.setVar("title", createPageTitle("User settings - " + this->userSession->user.login));
|
||||
Response result;
|
||||
result.setStatus(200);
|
||||
result.setBody(userSettingsPage.render());
|
||||
|
||||
return result;
|
||||
}
|
||||
|
||||
bool HandlerUserSettings::canAccess(const Permissions &perms)
|
||||
{
|
||||
return this->userSession->loggedIn;
|
||||
}
|
||||
|
||||
std::string HandlerUserSettings::accessErrorMessage()
|
||||
{
|
||||
return "Only logged-in users can change their settings";
|
||||
}
|
13
handlers/handlerusersettings.h
Normal file
13
handlers/handlerusersettings.h
Normal file
@ -0,0 +1,13 @@
|
||||
#ifndef HANDLERUSERSETTINGS_H
|
||||
#define HANDLERUSERSETTINGS_H
|
||||
#include "handler.h"
|
||||
class HandlerUserSettings : public Handler
|
||||
{
|
||||
public:
|
||||
using Handler::Handler;
|
||||
Response handleRequest(const Request &r);
|
||||
bool canAccess(const Permissions &perms);
|
||||
std::string accessErrorMessage();
|
||||
};
|
||||
|
||||
#endif // HANDLERUSERSETTINGS_H
|
Loading…
Reference in New Issue
Block a user