Fix win32 printing, fix my stupid json vulnerability

.gitea/workflows/tag.yaml

@@ -14,7 +14,7 @@ jobs:
     runs-on: arch-builder
     steps:
       - uses: actions/checkout@v4
-      - run: pacman --noconfirm -Sy mbedtls curl zip zig ninja
+      - run: pacman --noconfirm -Sy mbedtls zip zig ninja
       - run: pacman --config /opt/cross/ppc/pacman.conf --noconfirm -Sy mbedtls
       - run: cp Packaging/Linux/PKGBUILD /home/build/PKGBUILD
       - run: cp Packaging/Linux/build-arch.sh /home/build/build-arch.sh
@@ -22,7 +22,6 @@ jobs:
       - run: chown build:build /home/build/PKGBUILD
       - run: chown build:build /home/build/build-arch.sh
       - run: su build -c /home/build/build-arch.sh
-      - run: env -C Packaging/Tools bash build.sh
       - uses: akkuman/gitea-release-action@v1
         env:
           NODE_OPTIONS: '--experimental-fetch' # if nodejs < 18
@@ -33,11 +32,12 @@ jobs:
   
   update-tap-and-build-win32:
     name: "🔨 Build win32 and update the tap 🍺"
-    runs-on: global-containers-mingw
+    runs-on: global-container-mingw
     steps:
       - uses: actions/checkout@v4
       - run: |
-          env -C Builds/WinToolsIntel bash build.sh
+          mkdir artifacts
+          env -C Packaging/WinToolsIntel bash build.sh
 
       - uses: akkuman/gitea-release-action@v1
         env:

CMakeLists.txt

@@ -4,6 +4,7 @@ include(cmake/version.cmake)
 project(TessesFramework VERSION ${TESSESFRAMEWORK_MAJOR_VERSION}.${TESSESFRAMEWORK_MINOR_VERSION}.${TESSESFRAMEWORK_PATCH_VERSION})
 
 set(CMAKE_CXX_STANDARD 17)
+include(GNUInstallDirs)
 include(CheckLibraryExists)
 include(FetchContent)
 include(cmake/sources.cmake)

Packaging/Linux/PKGBUILD

@@ -1,6 +1,6 @@
 # Maintainer: Mike Nolan <tesses@tesses.net>
 pkgname=tessesframework # '-bzr', '-git', '-hg' or '-svn'
-pkgver=0.0.3
+pkgver=0.0.5
 pkgrel=1
 pkgdesc=""
 arch=('x86_64' 'powerpc')
@@ -10,7 +10,7 @@ groups=()
 depends=('mbedtls')
 makedepends=('git' 'cmake' 'make' 'base-devel' 'wget') # 'bzr', 'git', 'mercurial' or 'subversion'
 install=
-source=('tessesframework::git+https://git.tesses.org/tesses50/tessesframework')
+source=('tessesframework::git+https://git.tesses.org/tesses50/tessesframework#tag=v0.0.5')
 noextract=()
 sha256sums=('SKIP')
 if [[ -z "$CMAKE_TOOLCHAIN" ]]; then

Packaging/WinToolsIntel/x86.cmake

@@ -0,0 +1,7 @@
+set(CMAKE_C_COMPILER "/usr/bin/i686-w64-mingw32-gcc")
+set(CMAKE_CXX_COMPILER "/usr/bin/i686-w64-mingw32-g++")
+set(CMAKE_C_FLAGS "-static-libgcc -static-libstdc++ -static")
+set(CMAKE_CXX_FLAGS "-static-libgcc -static-libstdc++ -static")
+set(CMAKE_EXE_LINKER_FLAGS "-static-libgcc -static-libstdc++ -static")
+set(CMAKE_SYSTEM_NAME Windows)
+set(CMAKE_SYSTEM_PROCESSOR i686)

Packaging/WinToolsIntel/x86_64.cmake

@@ -0,0 +1,7 @@
+set(CMAKE_C_COMPILER "/usr/bin/x86_64-w64-mingw32-gcc")
+set(CMAKE_CXX_COMPILER "/usr/bin/x86_64-w64-mingw32-g++")
+set(CMAKE_C_FLAGS "-static-libgcc -static-libstdc++ -static")
+set(CMAKE_CXX_FLAGS "-static-libgcc -static-libstdc++ -static")
+set(CMAKE_EXE_LINKER_FLAGS "-static-libgcc -static-libstdc++ -static")
+set(CMAKE_SYSTEM_NAME Windows)
+set(CMAKE_SYSTEM_PROCESSOR x86_64)

Packaging/edit-formula.sh

@@ -5,7 +5,7 @@ echo "  desc \"\"" >> "Formula/tessesframework.rb"
 echo "  homepage \"\"" >> "Formula/tessesframework.rb"
 echo "  url \"https://git.tesses.org/tesses50/tessesframework/archive/$VERSION.tar.gz\"" >> "Formula/tessesframework.rb"
 echo "  sha256 \"$HASH\"" >> "Formula/tessesframework.rb"
-echo "  license \"MIT\"" >> "Formula/tessesframework.rb"
+echo "  license \"GPLv3\"" >> "Formula/tessesframework.rb"
 echo "  depends_on \"cmake\" => :build" >> "Formula/tessesframework.rb"
 echo "  depends_on \"mbedtls@3\"" >> "Formula/tessesframework.rb"
 echo "  def install" >> "Formula/tessesframework.rb"

changelog.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 0.0.5
+Fix win32 printing, fix my stupid json vulnerability
+
 ## 0.0.4
 Overhaul cmake configuration, add console api, fix http code that caused issues with cgi-bin
 

cmake/examples.cmake

@@ -1,4 +1,7 @@
 if(TESSESFRAMEWORK_ENABLE_EXAMPLES)
+        add_executable(start-my-website examples/start-my-website.cpp)
+        target_link_libraries(start-my-website PUBLIC tessesframework)
+
         add_executable(echo-my-name examples/echo-my-name.cpp)
         target_link_libraries(echo-my-name PUBLIC tessesframework)
 

cmake/version.cmake

@@ -1,3 +1,3 @@
 set(TESSESFRAMEWORK_MAJOR_VERSION 0)
 set(TESSESFRAMEWORK_MINOR_VERSION 0)
-set(TESSESFRAMEWORK_PATCH_VERSION 4)
+set(TESSESFRAMEWORK_PATCH_VERSION 5)

examples/start-my-website.cpp

@@ -0,0 +1,8 @@
+#include "TessesFramework/TessesFramework.hpp"
+
+int main(int argc, char **argv) {
+    Tesses::Framework::TF_Init();
+
+    Tesses::Framework::Platform::ShellFileOrUrl("https://tesses.net/");
+    return 0;
+}

include/TessesFramework/Console.hpp

@@ -20,7 +20,7 @@
 #include <cstdio>
 
 namespace Tesses::Framework {
-    enum class ConsoleColor {
+enum class ConsoleColor {
     CC_BLACK,
     CC_RED,
     CC_GREEN,
@@ -29,15 +29,15 @@ namespace Tesses::Framework {
     CC_MAGENTA,
     CC_CYAN,
     CC_WHITE
-    };
-    enum class ClearBehaviour {
+};
+enum class ClearBehaviour {
     CB_CURSORANDBELOW,
     CB_CURSORANDABOVE,
     CB_ENTIRESCREEN
-    };
-    struct Console {
+};
+struct Console {
     static bool IsTTY();
-        static std::pair<int,int> GetSize();
+    static std::pair<int, int> GetSize();
     static void ProgressBar(double);
     static void ProgressBar(int);
     static void Write(std::string text);
@@ -65,15 +65,16 @@ namespace Tesses::Framework {
     static void Clear();
     static void MoveToHome();
     static void ClearRetainPosition(ClearBehaviour clearBehaviour);
+    static void Reset();
+    static void SetInvertedColors(bool inverted);
     static void Flush();
 
-        //returns std::string::npos if no tty
-        static size_t List(std::vector<std::string>& strs);
-
+    // returns std::string::npos if no tty
+    static size_t List(std::vector<std::string> &strs);
 
     static std::string ReadPassword();
 
   private:
     static void WriteToStream(std::string_view text, bool isStderr);
-    };
-}
+};
+} // namespace Tesses::Framework

include/TessesFramework/Http/HttpClient.hpp

@@ -19,8 +19,10 @@
 #pragma once
 #include "../Streams/Stream.hpp"
 #include "HttpUtils.hpp"
+// clang-format off
 #include "TessesFramework/Filesystem/LocalFS.hpp"
 #include "TessesFramework/Filesystem/VFSFix.hpp"
+// clang-format on
 #include "WebSocket.hpp"
 namespace Tesses::Framework::Http {
 

include/TessesFramework/Platform/Environment.hpp

@@ -17,8 +17,10 @@
 */
 
 #pragma once
+// clang-format off
 #include "TessesFramework/Filesystem/LocalFS.hpp"
 #include "TessesFramework/Filesystem/VFSFix.hpp"
+// clang-format on
 #include <optional>
 namespace Tesses::Framework::Platform::Environment {
 extern const char EnvPathSeperator;

include/TessesFramework/Platform/Process.hpp

@@ -68,4 +68,6 @@ class Process {
 
     ~Process();
 };
+
+void ShellFileOrUrl(std::string fileOrUrl);
 } // namespace Tesses::Framework::Platform

src/Console.cpp

@@ -34,6 +34,58 @@
 
 namespace Tesses::Framework {
 
+class ConsoleHelpers {
+
+    bool canon;
+    bool echo;
+    bool signals;
+
+#if defined(_WIN32)
+    WORD colorFlags;
+
+#endif
+
+  public:
+    ConsoleHelpers() {
+        canon = Console::GetCanonical();
+        echo = Console::GetEcho();
+        signals = Console::GetSignals();
+#if defined(_WIN32)
+        HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
+        if (hConsole == INVALID_HANDLE_VALUE)
+            return;
+        CONSOLE_SCREEN_BUFFER_INFO bi;
+
+        GetConsoleScreenBufferInfo(hConsole, &bi);
+
+        colorFlags = bi.wAttributes;
+
+#endif
+    }
+
+    void reset() {
+#if defined(_WIN32)
+        HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
+        if (hConsole == INVALID_HANDLE_VALUE)
+            return;
+        SetConsoleTextAttribute(hConsole, colorFlags);
+#else
+        if (Console::IsTTY()) {
+            printf("\x1b[0m");
+            fflush(stdout);
+        }
+#endif
+    }
+
+    ~ConsoleHelpers() {
+        Console::SetCanonical(canon);
+        Console::SetEcho(echo);
+        Console::SetSignals(signals);
+        reset();
+    }
+};
+ConsoleHelpers consoleHelpers;
+
 size_t Console::List(std::vector<std::string> &strs) {
     if (!IsTTY())
         return std::string::npos;
@@ -54,11 +106,9 @@ size_t Console::List(std::vector<std::string> &strs) {
 
         for (int item = 0; item < size.second; ++item) {
             if (item == offsetInPage) {
-                Console::SetBackgroundColor(ConsoleColor::CC_WHITE, true);
-                Console::SetForegroundColor(ConsoleColor::CC_BLACK, false);
+                Console::SetInvertedColors(true);
             } else {
-                Console::SetBackgroundColor(ConsoleColor::CC_BLACK, false);
-                Console::SetForegroundColor(ConsoleColor::CC_WHITE, true);
+                Console::SetInvertedColors(false);
             }
 
             size_t myOffset = (size_t)item + page * (size.second);
@@ -72,8 +122,7 @@ size_t Console::List(std::vector<std::string> &strs) {
             }
         }
 
-        Console::SetBackgroundColor(ConsoleColor::CC_BLACK, false);
-        Console::SetForegroundColor(ConsoleColor::CC_WHITE, true);
+        Console::SetInvertedColors(false);
 
         int code = Console::Read();
 
@@ -102,22 +151,16 @@ size_t Console::List(std::vector<std::string> &strs) {
     }
     Console::SetEcho(echo);
     Console::SetCanonical(canonical);
-    Console::SetBackgroundColor(ConsoleColor::CC_BLACK, false);
-    Console::SetForegroundColor(ConsoleColor::CC_WHITE, true);
+    Console::SetInvertedColors(false);
     return i;
 }
 
-void Console::SetForegroundColor(ConsoleColor col, bool alt) {
+void Console::Reset() { consoleHelpers.reset(); }
 
-    if (!IsTTY())
-        return;
-#if __has_include(<termios.h>)
-    if (alt) {
-        printf("\x1b[%im", ((int)col) + 90); // this should be Write
-    } else {
-        printf("\x1b[%im", ((int)col) + 30); // this too
-    }
-#elif defined(_WIN32)
+#if defined(_WIN32)
+static bool vt100_inverted = false;
+static void _setcol(ConsoleColor col, bool alt, bool bg) {
+    if (bg) {
 
         HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
         if (hConsole == INVALID_HANDLE_VALUE)
@@ -126,55 +169,8 @@ void Console::SetForegroundColor(ConsoleColor col, bool alt) {
 
         GetConsoleScreenBufferInfo(hConsole, &bi);
 
-    bi.wAttributes &= ~(FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE |
-                        FOREGROUND_INTENSITY);
-
-    if (col == ConsoleColor::CC_RED || col == ConsoleColor::CC_YELLOW ||
-        col == ConsoleColor::CC_MAGENTA || col == ConsoleColor::CC_WHITE)
-        bi.wAttributes |= FOREGROUND_RED;
-
-    if (col == ConsoleColor::CC_GREEN || col == ConsoleColor::CC_YELLOW ||
-        col == ConsoleColor::CC_CYAN || col == ConsoleColor::CC_WHITE)
-        bi.wAttributes |= FOREGROUND_GREEN;
-
-    if (col == ConsoleColor::CC_BLUE || col == ConsoleColor::CC_CYAN ||
-        col == ConsoleColor::CC_MAGENTA || col == ConsoleColor::CC_WHITE)
-        bi.wAttributes |= FOREGROUND_BLUE;
-
-    if (alt)
-        bi.wAttributes |= FOREGROUND_INTENSITY;
-
-    SetConsoleTextAttribute(hConsole, bi.wAttributes);
-
-#else
-    if (alt) {
-        printf("\x1b[%im", ((int)col) + 90); // this should be Write
-    } else {
-        printf("\x1b[%im", ((int)col) + 30); // this too
-    }
-#endif
-}
-
-void Console::SetBackgroundColor(ConsoleColor col, bool alt) {
-    if (!IsTTY())
-        return;
-#if __has_include(<termios.h>)
-    if (alt) {
-        printf("\x1b[%im", ((int)col) + 100); // this should be Write
-    } else {
-        printf("\x1b[%im", ((int)col) + 40); // this too
-    }
-#elif defined(_WIN32)
-
-    HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
-    if (hConsole == INVALID_HANDLE_VALUE)
-        return;
-    CONSOLE_SCREEN_BUFFER_INFO bi;
-
-    GetConsoleScreenBufferInfo(hConsole, &bi);
-
-    bi.wAttributes &= ~(BACKGROUND_RED | BACKGROUND_GREEN | BACKGROUND_BLUE |
-                        BACKGROUND_INTENSITY);
+        bi.wAttributes &= ~(BACKGROUND_RED | BACKGROUND_GREEN |
+                            BACKGROUND_BLUE | BACKGROUND_INTENSITY);
 
         if (col == ConsoleColor::CC_RED || col == ConsoleColor::CC_YELLOW ||
             col == ConsoleColor::CC_MAGENTA || col == ConsoleColor::CC_WHITE)
@@ -192,6 +188,61 @@ void Console::SetBackgroundColor(ConsoleColor col, bool alt) {
             bi.wAttributes |= BACKGROUND_INTENSITY;
 
         SetConsoleTextAttribute(hConsole, bi.wAttributes);
+    } else {
+        HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
+        if (hConsole == INVALID_HANDLE_VALUE)
+            return;
+        CONSOLE_SCREEN_BUFFER_INFO bi;
+
+        GetConsoleScreenBufferInfo(hConsole, &bi);
+
+        bi.wAttributes &= ~(FOREGROUND_RED | FOREGROUND_GREEN |
+                            FOREGROUND_BLUE | FOREGROUND_INTENSITY);
+
+        if (col == ConsoleColor::CC_RED || col == ConsoleColor::CC_YELLOW ||
+            col == ConsoleColor::CC_MAGENTA || col == ConsoleColor::CC_WHITE)
+            bi.wAttributes |= FOREGROUND_RED;
+
+        if (col == ConsoleColor::CC_GREEN || col == ConsoleColor::CC_YELLOW ||
+            col == ConsoleColor::CC_CYAN || col == ConsoleColor::CC_WHITE)
+            bi.wAttributes |= FOREGROUND_GREEN;
+
+        if (col == ConsoleColor::CC_BLUE || col == ConsoleColor::CC_CYAN ||
+            col == ConsoleColor::CC_MAGENTA || col == ConsoleColor::CC_WHITE)
+            bi.wAttributes |= FOREGROUND_BLUE;
+
+        if (alt)
+            bi.wAttributes |= FOREGROUND_INTENSITY;
+
+        SetConsoleTextAttribute(hConsole, bi.wAttributes);
+    }
+}
+#endif
+
+void Console::SetForegroundColor(ConsoleColor col, bool alt) {
+
+    if (!IsTTY())
+        return;
+
+#if defined(_WIN32)
+
+    _setcol(col, alt, vt100_inverted);
+
+#else
+    if (alt) {
+        printf("\x1b[%im", ((int)col) + 90); // this should be Write
+    } else {
+        printf("\x1b[%im", ((int)col) + 30); // this too
+    }
+#endif
+}
+
+void Console::SetBackgroundColor(ConsoleColor col, bool alt) {
+    if (!IsTTY())
+        return;
+
+#if defined(_WIN32)
+    _setcol(col, alt, !vt100_inverted);
 #else
     if (alt) {
         printf("\x1b[%im", ((int)col) + 100); // this should be Write
@@ -201,6 +252,41 @@ void Console::SetBackgroundColor(ConsoleColor col, bool alt) {
 #endif
 }
 
+void Console::SetInvertedColors(bool inverted) {
+    if (!IsTTY())
+        return;
+#if defined(_WIN32)
+    if (inverted != vt100_inverted) {
+        HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
+        if (hConsole == INVALID_HANDLE_VALUE)
+            return;
+        CONSOLE_SCREEN_BUFFER_INFO bi;
+
+        GetConsoleScreenBufferInfo(hConsole, &bi);
+
+        bi.wAttributes =
+            (bi.wAttributes & 0xFF00) |
+            ((bi.wAttributes & (FOREGROUND_RED | FOREGROUND_GREEN |
+                                FOREGROUND_BLUE | FOREGROUND_INTENSITY))
+             << 4) |
+            ((bi.wAttributes & (BACKGROUND_RED | BACKGROUND_GREEN |
+                                BACKGROUND_BLUE | BACKGROUND_INTENSITY)) >>
+             4);
+
+        SetConsoleTextAttribute(hConsole, bi.wAttributes);
+    }
+
+    vt100_inverted = inverted;
+#else
+    if (inverted)
+        Write("\x1b[7m");
+    else
+        Write("\x1b[27m");
+
+    Flush();
+#endif
+}
+
 bool Console::SetEcho(bool echo) {
     if (!IsTTY())
         return false;
@@ -607,10 +693,18 @@ void Console::WriteToStream(std::string_view view, bool isStderr) {
                     }
 
                     if (text.size() > 0 && text.back() == 'm') {
-
+                        if (text.size() == 1) {
+                            consoleHelpers.reset();
+                        } else {
                             try {
                                 auto num =
                                     std::stol(text.substr(0, text.size() - 1));
+                                if (num == 0)
+                                    consoleHelpers.reset();
+                                if (num == 7)
+                                    Console::SetInvertedColors(true);
+                                if (num == 27)
+                                    Console::SetInvertedColors(false);
                                 if (num >= 30 && num <= 37) {
                                     SetForegroundColor((ConsoleColor)(num - 30),
                                                        false);
@@ -621,12 +715,13 @@ void Console::WriteToStream(std::string_view view, bool isStderr) {
                                     SetForegroundColor((ConsoleColor)(num - 90),
                                                        true);
                                 } else if (num >= 100 && num <= 107) {
-                                SetBackgroundColor((ConsoleColor)(num - 100),
-                                                   true);
+                                    SetBackgroundColor(
+                                        (ConsoleColor)(num - 100), true);
                                 }
                             } catch (...) {
                             }
                         }
+                    }
 
                     text.clear();
 

src/Filesystem/LocalFS.cpp

@@ -24,9 +24,10 @@
 #include <sys/stat.h>
 #include <sys/types.h>
 #if defined(_WIN32)
-
-#include "TessesFramework/Filesystem/VFSFix.hpp"
+// clang-format off
 #include <windows.h>
+#include "TessesFramework/Filesystem/VFSFix.hpp"
+// clang format on
 #undef min
 #else
 #include <sys/statvfs.h>

src/Http/HttpServer.cpp

@@ -19,8 +19,7 @@
 #include "TessesFramework/Http/HttpServer.hpp"
 #include "TessesFramework/Common.hpp"
 #include "TessesFramework/Crypto/Crypto.hpp"
-#include "TessesFramework/Filesystem/VFS.hpp"
-#include "TessesFramework/Filesystem/VFSFix.hpp"
+
 #include "TessesFramework/Http/ContentDisposition.hpp"
 #include "TessesFramework/Http/HttpStream.hpp"
 #include "TessesFramework/Streams/BufferedStream.hpp"
@@ -33,6 +32,10 @@
 
 #include <TessesFramework/Http/HttpUtils.hpp>
 #include <iostream>
+// clang-format off
+#include "TessesFramework/Filesystem/VFS.hpp"
+#include "TessesFramework/Filesystem/VFSFix.hpp"
+// clang-format on
 using FileStream = Tesses::Framework::Streams::FileStream;
 using Stream = Tesses::Framework::Streams::Stream;
 using SeekOrigin = Tesses::Framework::Streams::SeekOrigin;
@@ -711,8 +714,8 @@ void HttpServer::StartAccepting() {
     }
     if (this->showARTL) {
         if (!svr->IsValid())
-            StdOut()
-                << "\x1B[31mError, we failed to bind or something\x1B[39m" << NewLine();
+            StdOut() << "\x1B[31mError, we failed to bind or something\x1B[39m"
+                     << NewLine();
         StdOut() << "\x1B[31mAlmost Ready to Listen\x1B[39m" << NewLine();
     }
 

src/Platform/Environment.cpp

@@ -22,9 +22,12 @@
 #include "sago/platform_folders.h"
 #endif
 #if defined(_WIN32)
-#include "TessesFramework/Filesystem/VFSFix.hpp"
-#include "TessesFramework/Text/StringConverter.hpp"
+
+// clang-format off
 #include <windows.h>
+#include "TessesFramework/Text/StringConverter.hpp"
+#include "TessesFramework/Filesystem/VFSFix.hpp"
+// clang-format on
 
 using namespace Tesses::Framework::Text::StringConverter;
 #endif

src/Platform/Process.cpp

@@ -691,4 +691,38 @@ std::shared_ptr<Tesses::Framework::Streams::Stream> Process::GetStderrStream() {
 #endif
 }
 
+void ShellFileOrUrl(std::string fileOrUrl) {
+#if defined(GEKKO) || defined(__PS2__) || defined(__SWITCH__)
+    throw std::runtime_error("Platform not supported");
+#elif !defined(TESSESFRAMEWORK_ENABLE_PROCESS)
+    throw std::runtime_error("Process not enabled");
+#elif defined(_WIN32)
+    auto exec = Tesses::Framework::Platform::Environment::GetRealExecutablePath(
+        (std::string) "cmd");
+    Process p(exec.ToString(), {"cmd", "/c", "start", fileOrUrl});
+    if (p.Start())
+        if (p.WaitForExit() != 0)
+            throw std::runtime_error("Exit code did not indicate success");
+#elif defined(__APPLE__)
+    auto exec = Tesses::Framework::Platform::Environment::GetRealExecutablePath(
+        (std::string) "open");
+    Process p(exec.ToString(), {"open", fileOrUrl});
+    if (p.Start())
+        if (p.WaitForExit() != 0)
+            throw std::runtime_error("Exit code did not indicate success");
+#else
+    if (fileOrUrl == "--help" || fileOrUrl == "--version" ||
+        fileOrUrl == "--manual")
+        throw std::invalid_argument(
+            "Invalid argument \"" + fileOrUrl +
+            "\", this function expects a file or url not xdg-open flags");
+    auto exec = Tesses::Framework::Platform::Environment::GetRealExecutablePath(
+        (std::string) "xdg-open");
+    Process p(exec.ToString(), {"xdg-open", fileOrUrl});
+    if (p.Start())
+        if (p.WaitForExit() != 0)
+            throw std::runtime_error("Exit code did not indicate success");
+#endif
+}
+
 } // namespace Tesses::Framework::Platform

src/Serialization/Json.cpp

@@ -15,9 +15,9 @@
     You should have received a copy of the GNU General Public License
     along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
-
 #include "TessesFramework/Serialization/Json.hpp"
 #include "TessesFramework/Http/HttpUtils.hpp"
+#include "TessesFramework/TessesFramework.hpp"
 #include <functional>
 
 #include "TessesFramework/Common.hpp"
@@ -129,6 +129,7 @@ JToken Json::Decode(std::string str) {
                                     v |= HttpUtils::HexToNibble(str[i])
                                          << ((3 - i2) * 4);
                                 }
+                                i--;
 
                                 uint32_t v2 = v;
 
@@ -274,7 +275,8 @@ JToken Json::Decode(std::string str) {
                         break;
                     } else {
 
-                        throw std::runtime_error("Invalid JSON object.");
+                        throw std::runtime_error("Invalid JSON object. " +
+                                                 std::to_string(tokenIndex));
                     }
                 }
                 return obj;