Styling, Certificates, and SSL_Utils

- Added clang-tidy config file + CI
- Reformated the code to fit the new Style
- Added SSL_Utils for better smart pointers of SSL objects
- Added Certificate class + extras to manipulate Certificates

Author: f-michaut

.clang-tidy

@@ -0,0 +1,38 @@
+Checks: '
+*,
+-llvmlibc-*,
+-google-readability-todo,
+-fuchsia-default-arguments-calls,
+-fuchsia-default-arguments-declarations,
+-cppcoreguidelines-init-variables,
+-cppcoreguidelines-pro-type-reinterpret-cast,
+-android-*,
+-altera-*,
+-llvm-namespace-comment,
+-readability-implicit-bool-conversion,
+-google-explicit-constructor,
+-fuchsia-overloaded-operator,
+
+-google-readability-braces-around-statements,
+-google-readability-namespace-comments,
+-hicpp-braces-around-statements,
+-hicpp-explicit-conversions,
+-fuchsia-trailing-return
+'
+
+WarningsAsErrors: 'bugprone-exception-escape'
+FormatStyle: 'none' # TODO: Replace with 'file' once we have a proper .clang-format file
+InheritParentConfig: true
+CheckOptions:
+  misc-include-cleaner.MissingIncludes: 'false'
+  misc-include-cleaner.IgnoreHeaders: 'CppSockets/OSDetection\.hpp'
+
+  bugprone-argument-comment.StrictMode: 1
+
+  # Readability
+  readability-braces-around-statements.ShortStatementLines: 2
+
+  readability-identifier-naming.NamespaceCase: CamelCase
+
+  readability-identifier-length.IgnoredVariableNames: "^(fd|nb|ss)$"
+  readability-identifier-length.IgnoredParameterNames: "^([n]|fd)$"

.editorconfig

@@ -0,0 +1,11 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+
+[*.{c++,cc,cpp,cxx,h,h++,hh,hpp,hxx,inl,ipp,tlh,tli}]
+cpp_indent_case_contents_when_block = true
+cpp_new_line_before_open_brace_namespace = same_line
+indent_size = 4
+indent_style = space

.github/workflows/ci.yml

@@ -0,0 +1,19 @@
+name: C++ CI
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+    types: [ "opened", "reopened", "synchronize", "ready_for_review" ]
+
+jobs:
+  clang-tidy:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v4
+
+      - name: clang-tidy review
+        uses: ZedThree/clang-tidy-review@v0.21.0

.github/workflows/cmake-multi-platform.yml

@@ -4,14 +4,16 @@ on:
   push:
     branches: [ "master" ]
   pull_request:
+    branches: [ "master" ]
     types: [ "opened", "reopened", "synchronize", "ready_for_review" ]
 
 jobs:
   build:
     runs-on: ${{ matrix.os }}
 
     strategy:
-      fail-fast: false # ensure we don't stop after 1 failure to always have a complete picture of what is failing
+      # ensure we don't stop after 1 failure to always have a complete picture of what is failing
+      fail-fast: false
 
       # Set up a matrix to run the following configurations:
       # - ubuntu Debug/Release clang/gcc
@@ -50,7 +52,8 @@ jobs:
     - uses: actions/checkout@v3
 
     - name: Set reusable strings
-      # Turn repeated input strings (such as the build output directory) into step outputs. These step outputs can be used throughout the workflow file.
+      # Turn repeated input strings (such as the build output directory) into step outputs.
+      # These step outputs can be used throughout the workflow file.
       id: strings
       shell: bash
       run: |
@@ -72,7 +75,8 @@ jobs:
         vcpkg install
 
     - name: Configure CMake
-      # Configure CMake in a 'build' subdirectory. `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make.
+      # Configure CMake in a 'build' subdirectory.
+      # `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make.
       # See https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html?highlight=cmake_build_type
       run: >
         cmake -B ${{ steps.strings.outputs.build-output-dir }}
@@ -83,11 +87,13 @@ jobs:
         -S ${{ github.workspace }}
 
     - name: Build
-      # Build your program with the given configuration. Note that --config is needed because the default Windows generator is a multi-config generator (Visual Studio generator).
+      # Build your program with the given configuration. Note that --config is needed
+      # because the default Windows generator is a multi-config generator (Visual Studio generator).
       run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --config ${{ matrix.build_type }}
 
     - name: Test
       working-directory: ${{ steps.strings.outputs.build-output-dir }}
-      # Execute tests defined by the CMake configuration. Note that --build-config is needed because the default Windows generator is a multi-config generator (Visual Studio generator).
+      # Execute tests defined by the CMake configuration. Note that --build-config is needed
+      # because the default Windows generator is a multi-config generator (Visual Studio generator).
       # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
-      run: ctest --build-config ${{ matrix.build_type }} --test-dir tests
+      run: ctest --build-config ${{ matrix.build_type }} --test-dir tests --output-on-failure

.gitignore

@@ -1,6 +1,11 @@
 build/
 
+libcppsockets.so
 compile_commands.json
 
 *.tmp
 *.gch
+vgcore.*
+
+.vscode/
+.cache/

CMakeLists.txt

@@ -4,7 +4,7 @@
 ## Author Francois Michaut
 ##
 ## Started on  Sun Aug 28 19:26:51 2022 Francois Michaut
-## Last update Sat Dec  2 17:45:28 2023 Francois Michaut
+## Last update Mon Aug  4 23:34:08 2025 Francois Michaut
 ##
 ## CMakeLists.txt : CMake to build the CppSockets library
 ##
@@ -22,7 +22,9 @@ configure_file(include/CppSockets/Version.hpp.in include/CppSockets/Version.hpp)
 
 add_library(cppsockets
   source/Address.cpp
+  source/Certificate.cpp
   source/IPv4.cpp
+  source/SSL_Utils.cpp
   source/Socket.cpp
   source/SocketInit.cpp
   source/TlsSocket.cpp

include/CppSockets/Address.hpp

@@ -20,19 +20,19 @@
 namespace CppSockets {
     class IAddress {
         public:
-            [[nodiscard]] virtual std::uint32_t getAddress() const = 0;
-            [[nodiscard]] virtual int getFamily() const = 0;
-            [[nodiscard]] virtual const std::string &toString() const = 0;
+            [[nodiscard]] virtual auto getAddress() const -> std::uint32_t = 0;
+            [[nodiscard]] virtual auto getFamily() const -> int = 0;
+            [[nodiscard]] virtual auto toString() const -> const std::string & = 0;
     };
 
     class IEndpoint {
         public:
-            [[nodiscard]] virtual std::uint16_t getPort() const = 0;
-            [[nodiscard]] virtual const IAddress &getAddr() const = 0;
-            [[nodiscard]] virtual const std::string &toString() const = 0;
+            [[nodiscard]] virtual auto getPort() const -> std::uint16_t = 0;
+            [[nodiscard]] virtual auto getAddr() const -> const IAddress & = 0;
+            [[nodiscard]] virtual auto toString() const -> const std::string & = 0;
 
         protected:
-            [[nodiscard]] std::string makeString() const;
+            [[nodiscard]] auto makeString() const -> std::string;
     };
 
     template <class T>
@@ -46,15 +46,15 @@ namespace CppSockets {
             {};
             virtual ~Endpoint() = default;
 
-            [[nodiscard]] std::uint16_t getPort() const override {
+            [[nodiscard]] auto getPort() const -> std::uint16_t override {
                 return port;
             }
 
-            [[nodiscard]] const T &getAddr() const override {
+            [[nodiscard]] auto getAddr() const -> const T & override {
                 return addr;
             }
 
-            [[nodiscard]] const std::string &toString() const override {
+            [[nodiscard]] auto toString() const -> const std::string & override {
                 return str;
             }
         private: