scheduler canceling, sessiion query, didn't test yet!

database/exceptions.cpp

@@ -18,3 +18,7 @@ DB::EmptyResult::EmptyResult(const std::string& text):
 DB::NoLogin::NoLogin(const std::string& text):
     EmptyResult(text)
 {}
+
+DB::NoSession::NoSession (const std::string& text):
+    EmptyResult(text)
+{}

database/exceptions.h

@@ -26,4 +26,9 @@ class NoLogin : public EmptyResult {
 public:
     explicit NoLogin(const std::string& text);
 };
+
+class NoSession : public EmptyResult {
+public:
+    explicit NoSession(const std::string& text);
+};
 }

database/interface.h

@@ -9,6 +9,13 @@
 #include <stdint.h>
 
 namespace DB {
+struct Session {
+    unsigned int id;
+    unsigned int owner;
+    std::string accessToken;
+    std::string renewToken;
+};
+
 class Interface {
 public:
     enum class Type {
@@ -40,6 +47,7 @@ public:
     virtual unsigned int registerAccount(const std::string& login, const std::string& hash) = 0;
     virtual std::string getAccountHash(const std::string& login) = 0;
     virtual unsigned int createSession(const std::string& login, const std::string& access, const std::string& renew) = 0;
+    virtual Session findSession(const std::string& accessToken) = 0;
 
 protected:
     Interface(Type type);

database/mysql/mysql.cpp

@@ -20,6 +20,7 @@ constexpr const char* assignRoleQuery = "INSERT INTO roleBindings (`account`, `r
 constexpr const char* selectHash = "SELECT password FROM accounts where login = ?";
 constexpr const char* createSessionQuery = "INSERT INTO sessions (`owner`, `access`, `renew`, `persist`, `device`)"
                                         " SELECT accounts.id, ?, ?, true, ? FROM accounts WHERE accounts.login = ?";
+constexpr const char* selectSession = "SELECT id, owner, renew FROM sessions where access = ?";
 
 static const std::filesystem::path buildSQLPath = "database";
 
@@ -298,6 +299,24 @@ unsigned int DB::MySQL::lastInsertedId() {
     else
         throw std::runtime_error(std::string("Querying last inserted id returned no rows"));
 }
-
+DB::Session DB::MySQL::findSession(const std::string& accessToken) {
+    std::string a = accessToken;
+    MYSQL* con = &connection;
+
+    Statement session(con, selectSession);
+    session.bind(a.data(), MYSQL_TYPE_STRING);
+
+    std::vector<std::vector<std::any>> result = session.fetchResult();
+    if (result.empty())
+        throw NoSession("Couldn't find session with token " + a);
+
+    DB::Session res;
+    res.id = std::any_cast<unsigned int>(result[0][0]);
+    res.owner = std::any_cast<unsigned int>(result[0][1]);
+    res.renewToken = std::any_cast<const std::string&>(result[0][2]);
+    res.accessToken = a;
+
+    return res;
+}
 
 

database/mysql/mysql.h

@@ -34,6 +34,7 @@ public:
     unsigned int registerAccount(const std::string& login, const std::string& hash) override;
     std::string getAccountHash(const std::string& login) override;
     unsigned int createSession(const std::string& login, const std::string& access, const std::string& renew) override;
+    Session findSession(const std::string& accessToken) override;
 
 private:
     void executeFile(const std::filesystem::path& relativePath);

handler/CMakeLists.txt

@@ -7,6 +7,7 @@ set(HEADERS
     env.h
     register.h
     login.h
+    poll.h
 )
 
 set(SOURCES
@@ -15,6 +16,7 @@ set(SOURCES
     env.cpp
     register.cpp
     login.cpp
+    poll.cpp
 )
 
 target_sources(${PROJECT_NAME} PRIVATE ${SOURCES})

handler/poll.cpp

@@ -6,6 +6,7 @@
 #include "response/response.h"
 #include "server/server.h"
 #include "request/redirect.h"
+#include "database/exceptions.h"
 
 Handler::Poll::Poll (Server* server):
     Handler("login", Request::Method::get),
@@ -25,6 +26,8 @@ void Handler::Poll::handle (Request& request) {
         throw Redirect(&session);
     } catch (const Redirect& r) {
         throw r;
+    } catch (const DB::NoSession& e) {
+        return error(request, Result::tokenProblem, Response::Status::unauthorized);
     } catch (const std::exception& e) {
         std::cerr << "Exception on poll:\n\t" << e.what() << std::endl;
         return error(request, Result::unknownError, Response::Status::internalError);

server/server.cpp

@@ -192,6 +192,22 @@ Session& Server::openSession(const std::string& login) {
     if (sessionId == 0)
         throw std::runtime_error("Couldn't create session, ran out of attempts");
 
-    std::unique_ptr<Session>& session = sessions[accessToken] = std::make_unique<Session>(sessionId, accessToken, renewToken);
+    std::unique_ptr<Session>& session = sessions[accessToken] = std::make_unique<Session>(scheduler, sessionId, accessToken, renewToken);
+    return *session.get();
+}
+
+Session& Server::getSession (const std::string& accessToken) {
+    Sessions::const_iterator itr = sessions.find(accessToken);
+    if (itr != sessions.end())
+        return *(itr->second);
+
+    DB::Resource db = pool->request();
+    DB::Session s = db->findSession(accessToken);
+    std::unique_ptr<Session>& session = sessions[accessToken] = std::make_unique<Session>(
+        scheduler,
+        s.id,
+        s.accessToken,
+        s.renewToken
+    );
     return *session.get();
 }

server/session.cpp

@@ -15,9 +15,17 @@ Session::Session(
     id(id),
     access(access),
     renew(renew),
-    polling(nullptr)
+    polling(nullptr),
+    timeoutId(TM::Scheduler::none)
 {}
 
+Session::~Session () {
+    if (timeoutId != TM::Scheduler::none) {
+        if (std::shared_ptr<TM::Scheduler> sch = scheduler.lock())
+            sch->cancel(timeoutId);
+    }
+}
+
 std::string Session::getAccessToken() const {
     return access;
 }
@@ -27,18 +35,30 @@ std::string Session::getRenewToken() const {
 }
 
 void Session::accept(std::unique_ptr<Request> request) {
+    std::shared_ptr<TM::Scheduler> sch = scheduler.lock();
     if (polling) {
         Handler::Poll::error(*request.get(), Handler::Poll::Result::replace, Response::Status::ok);
+        if (timeoutId != TM::Scheduler::none) {
+            if (sch)
+                sch->cancel(timeoutId);
+
+            timeoutId = TM::Scheduler::none;
+        }
         //TODO unschedule
     }
 
-    std::shared_ptr<TM::Scheduler> sch = scheduler.lock();
     if (!sch) {
         std::cerr << "Was unable to schedule polling timeout, replying with an error" << std::endl;
         Handler::Poll::error(*request.get(), Handler::Poll::Result::unknownError, Response::Status::internalError);
         return;
     }
-    sch->schedule(std::bind(&Session::onTimeout, this), TM::Scheduler::Delay(5000));
+    timeoutId = sch->schedule(std::bind(&Session::onTimeout, this), TM::Scheduler::Delay(5000));
 
     polling = std::move(request);
 }
+
+void Session::onTimeout () {
+    timeoutId = TM::Scheduler::none;
+    Handler::Poll::error(*polling.get(), Handler::Poll::Result::timeout, Response::Status::ok);
+    polling.reset();
+}

server/session.h

@@ -18,6 +18,7 @@ public:
     );
     Session(const Session&) = delete;
     Session(Session&& other);
+    ~Session();
     Session& operator = (const Session&) = delete;
     Session& operator = (Session&& other);
 
@@ -34,4 +35,5 @@ private:
     std::string access;
     std::string renew;
     std::unique_ptr<Request> polling;
+    TM::Record::ID timeoutId;
 };

taskmanager/CMakeLists.txt

@@ -7,6 +7,7 @@ set(HEADERS
     route.h
     scheduler.h
     function.h
+    record.h
 )
 
 set(SOURCES
@@ -15,6 +16,7 @@ set(SOURCES
     route.cpp
     scheduler.cpp
     function.cpp
+    record.cpp
 )
 
 target_sources(${PROJECT_NAME} PRIVATE ${SOURCES})

taskmanager/record.cpp

@@ -0,0 +1,18 @@
+//SPDX-FileCopyrightText: 2024 Yury Gubich <blue@macaw.me>
+//SPDX-License-Identifier: GPL-3.0-or-later
+
+#include "record.h"
+
+TM::Record::Record (ID id, const Task& task, Time time):
+    id(id),
+    task(task),
+    time(time)
+{}
+
+bool TM::Record::operator < (const Record& other) const {
+    return time < other.time;
+}
+
+bool TM::Record::operator > (const Record& other) const {
+    return time > other.time;
+}

taskmanager/record.h

@@ -0,0 +1,26 @@
+//SPDX-FileCopyrightText: 2024 Yury Gubich <blue@macaw.me>
+//SPDX-License-Identifier: GPL-3.0-or-later
+
+#pragma once
+
+#include <cstdint>
+#include <chrono>
+#include <functional>
+
+namespace TM {
+class Record {
+public:
+    using Time = std::chrono::time_point<std::chrono::steady_clock>;
+    using Task = std::function<void()>;
+    using ID = uint64_t;
+
+    Record(ID id, const Task& task, Time time);
+
+    ID id;
+    Task task;
+    Time time;
+
+    bool operator > (const Record& other) const;
+    bool operator < (const Record& other) const;
+};
+}

taskmanager/scheduler.cpp

@@ -3,13 +3,17 @@
 
 #include "scheduler.h"
 
+const TM::Record::ID TM::Scheduler::none = 0;
+
 TM::Scheduler::Scheduler (std::weak_ptr<Manager> manager):
     queue(),
+    scheduled(),
     manager(manager),
     mutex(),
     cond(),
     thread(nullptr),
-    running(false)
+    running(false),
+    idCounter(TM::Scheduler::none)
 {}
 
 TM::Scheduler::~Scheduler () {
@@ -50,28 +54,39 @@ void TM::Scheduler::loop () {
 
         Time currentTime = std::chrono::steady_clock::now();
         while (!queue.empty()) {
-            Time nextScheduledTime = queue.top().first;
+            Time nextScheduledTime = queue.top().time;
             if (nextScheduledTime > currentTime) {
                 cond.wait_until(lock, nextScheduledTime);
                 break;
             }
 
-            Record task = queue.pop();
+            Record record = queue.pop();
+            std::size_t count = scheduled.erase(record.id);
+            if (count == 0)     //it means this record has been cancelled, no need to execute it
+                continue;
+
             lock.unlock();
             std::shared_ptr<Manager> mngr = manager.lock();
             if (mngr)
-                mngr->schedule(std::move(task.second));
+                mngr->schedule(std::make_unique<Function>(record.task));
 
             lock.lock();
         }
     }
 }
 
-void TM::Scheduler::schedule (const std::function<void()>& task, Delay delay) {
+TM::Record::ID TM::Scheduler::schedule (const Task& task, Delay delay) {
     std::unique_lock lock(mutex);
     Time time = std::chrono::steady_clock::now() + delay;
-    queue.emplace(time, std::make_unique<Function>(task));
+    queue.emplace(++idCounter, task, time);
+    scheduled.emplace(idCounter);
 
     lock.unlock();
     cond.notify_one();
+
+    return idCounter;
 }
+
+bool TM::Scheduler::cancel (Record::ID id) {
+    return scheduled.erase(id) != 0;        //not to mess with the queue, here we just mark it as not scheduled
+}                                           //and when the time comes it will be just discarded

taskmanager/scheduler.h

@@ -6,43 +6,47 @@
 #include <memory>
 #include <thread>
 #include <chrono>
-#include <functional>
 #include <mutex>
 #include <condition_variable>
+#include <set>
 
 #include "manager.h"
 #include "function.h"
+#include "record.h"
 #include "utils/helpers.h"
 
 namespace TM {
 class Scheduler {
 public:
     using Delay = std::chrono::milliseconds;
+    using Time = Record::Time;
+    using Task = Record::Task;
 
     Scheduler (std::weak_ptr<Manager> manager);
     ~Scheduler ();
 
-    void start();
-    void stop();
-    void schedule(const std::function<void()>& task, Delay delay);
+    void start ();
+    void stop ();
+    Record::ID schedule (const Task& task, Delay delay);
+    bool cancel (Record::ID id);
+
+    static const Record::ID none;
 
 private:
-    void loop();
+    void loop ();
 
 private:
-    using Task = std::unique_ptr<Function>;
-    using Time = std::chrono::time_point<std::chrono::steady_clock>;
-    using Record = std::pair<Time, Task>;
-
     PriorityQueue<
         Record,
         std::vector<Record>,
-        FirstGreater<Record>
+        std::greater<>
     > queue;
+    std::set<Record::ID> scheduled;
     std::weak_ptr<Manager> manager;
     std::mutex mutex;
     std::condition_variable cond;
     std::unique_ptr<std::thread> thread;
     bool running;
+    Record::ID idCounter;
 };
 }