Add new endpoint & update api-logs

tklein1801 · Aug 1, 2021 · ddb2d28 · ddb2d28
1 parent 0f389cf
commit ddb2d28
Show file tree

Hide file tree

Showing 6 changed files with 50 additions and 7 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -1,5 +1,18 @@
 ## Changelogs
 
+### [BBS-Mitfahrzentrale v0.5.1-pre](https://github.com/tklein1801/BBS-Mitfahrzentrale/releases/tag/v0.5.1-pre)
+
+#### Hinzugefügt
+
+- Endpunkt um zu überprüfen ob die Datenbankverbindung erfolgreich war oder nicht
+
+#### Geändert
+
+- Die Anfragemethode wird nun auch bei den API-Logs gespeichert
+- Das Datum bei API-Logs wurd nun angezeigt
+
+---
+
 ### [BBS-Mitfahrzentrale v0.5-pre](https://github.com/tklein1801/BBS-Mitfahrzentrale/releases/tag/v0.5-pre)
 
 #### Hinzugefügt

diff --git a/assets/doc/API.md b/assets/doc/API.md
@@ -31,6 +31,12 @@
 
 ---
 
+## database
+
+> Wird verwendet um zu prüfen ob die Datenbankverbindung erfolgreich war. Sollte die Verbindung scheitern wird hier die Fehlermeldung angezeigt. Sollte keine Fehlermeldung angezeigt werden ist davon auszugehen dass die Verbindung erfolgreich war
+
+---
+
 ## user
 
 **Registrieren eines neuen Benutzers**

diff --git a/assets/php/ApiLogger.php b/assets/php/ApiLogger.php
@@ -3,12 +3,12 @@
 
 class ApiLogger 
 {  
-  public function create(int $responseCode, string $requestedPath, string $requestedIp, $requestedKey)
+  public function create(int $responseCode, string $requestedPath, string $requestedIp, string $requestMethod, $requestedKey)
   {
     require get_defined_constants()['CON_PATH'];
 
-    $insert = $con->prepare("INSERT INTO `cshare_apiLogs` (`response_code`, `requestedPath`, `requestedIp`, `requestKey`) VALUES (?, ?, ?, ?)");
-    $insert->bind_param("isss", $responseCode, $requestedPath, $requestedIp, $requestedKey);
+    $insert = $con->prepare("INSERT INTO `cshare_apiLogs` (`response_code`, `requestedPath`, `requestedIp`, `requestMethod`, `requestKey`) VALUES (?, ?, ?, ?, ?)");
+    $insert->bind_param("issss", $responseCode, $requestedPath, $requestedIp, $requestMethod, $requestedKey);
     $insert->execute();
 
     return $insert;
@@ -93,6 +93,8 @@ public function createLog($requestedPath)
 
     $HTTP_STATUS_CODE = http_response_code();
 
-    $this->create($HTTP_STATUS_CODE, $requestedPath, $clientIp, $apiKey);
+    $requestMethod = $_SERVER['REQUEST_METHOD'];
+
+    $this->create($HTTP_STATUS_CODE, $requestedPath, $clientIp, $requestMethod, $apiKey);
   }
 }
diff --git a/endpoints/sql.php b/endpoints/sql.php
@@ -1,5 +1,5 @@
 <?php
-$con = new mysqli("HOST", "USERNAME", "PASSWORD", "bbs-mitfahrzentrale");
+$con = new mysqli("dulliag.de", "db_thorben", "Pizza123", "bbs-mitfahrzentrale-v2");
 $con->set_charset("utf8");
 if($con->connect_error) {
 	echo "Connection to database failed. Reason: <br>";

diff --git a/index.php b/index.php
@@ -28,7 +28,7 @@
 date_default_timezone_set("Europe/Berlin");
 
 // Application information
-define('CURRENT_APP_VERSION', 'v0.5-pre');
+define('CURRENT_APP_VERSION', 'v0.5.1-pre');
 define('GITHUB', array('author' => 'Thorben Klein',
   'user' => 'tklein1801',
   'repo' => 'BBS-Mitfahrzentrale'
@@ -258,6 +258,13 @@ function getApiKey() {
 });
 
 // Avaiable endpoints
+/**
+ * Database connection
+ */
+Route::add($GLOBALS['apiPath'] . "database", function () {
+  require_once "endpoints/sql.php";
+});
+
 /**
  * User
  */

diff --git a/routes/admin/logs.php b/routes/admin/logs.php
@@ -48,8 +48,10 @@
                           <thead>
                             <tr>
                               <th class="text-center">ID</th>
+                              <th>Datum</th>
                               <th class="text-center">Statuscode</th>
-                              <th>Pfad</th>
+                              <th>Methode</th>
+                              <th class="text-center">Pfad</th>
                               <th>IP-Adresse</th>
                               <th colspan="2">API-Key</th>
                             </tr>
@@ -59,6 +61,7 @@
                               if ($LOG_COUNT > 0) {
                                 foreach ($logs as $log) {
                                   $statusCode = $log['response_code'];
+                                  $requestMethod = $log['requestMethod'];
                                   if ($statusCode >= 500 && $statusCode < 600) {
                                     $statusBadge = '<span class="badge bg-danger">'.$statusCode.'</span>';
                                   } else if ($statusCode >= 400) {
@@ -70,12 +73,24 @@
                                   } else {
                                     $statusBadge = '<span class="badge bg-secondary">'.$statusCode.'</span>';
                                   }
+
+                                  if ($requestMethod !== null) {
+                                    $requestMethodBadge = '<span class="badge bg-warning">'.$requestMethod.'</span>';                                    
+                                  } else {
+                                    $requestMethodBadge = '<span class="badge bg-warning">NULL</span>';
+                                  }
                                   echo '<tr>
                                     <td class="text-center">
                                       <p class="text-center">'.$log['logId'].'</p>
                                     </td>
+                                    <td class="text-center">
+                                      '.date("d.m.Y • H:i", strtotime($log['requestedAt'])).'
+                                    </td>
                                     <td class="text-center">
                                       '.$statusBadge.'
+                                    </td>     
+                                    <td class="text-center">
+                                      '.$requestMethodBadge.'
                                     </td>
                                     <td>
                                       <p>'.$log['requestedPath'].'</p>