fix check and implement check/delete

Author: rdeutz

app/Http/Controllers/Api/V1/SiteController.php

@@ -33,7 +33,7 @@ public function register(Request $request): JsonResponse
         $url = $request->string('url');
         $key = $request->string('key');
 
-        if ($url->isEmpty() && $key->isEmpty()) {
+        if ($url->isEmpty() || $key->isEmpty()) {
             return $this->error('BadRequest');
         }
 
@@ -72,7 +72,25 @@ public function register(Request $request): JsonResponse
      */
     public function check(Request $request): JsonResponse
     {
-        return response()->json(['check']);
+        $url = $request->string('url');
+        $key = $request->string('key');
+
+        if ($url->isEmpty() || $key->isEmpty()) {
+            return $this->error('BadRequest');
+        }
+
+        $connectionService = new Connection($url, $key);
+
+        // Do a health check
+        try {
+            $connectionService->checkHealth();
+        } catch (ServerException $e) {
+            return $this->error($e->getMessage(), 500);
+        } catch (ClientException|\Exception $e) {
+            return $this->error($e->getMessage());
+        }
+
+        return $this->ok();
     }
 
     /**
@@ -82,6 +100,20 @@ public function check(Request $request): JsonResponse
      */
     public function delete(Request $request): JsonResponse
     {
-        return response()->json(['delete']);
+        $url = $request->string('url');
+        $key = $request->string('key');
+
+        if ($url->isEmpty() || $key->isEmpty()) {
+            return $this->error('BadRequest');
+        }
+
+        try {
+            Site::where('url', $url)->where('key', $key)->delete();
+        }
+        catch (\Exception $e) {
+            return $this->error($e->getMessage());
+        }
+
+        return $this->ok();
     }
 }

routes/api.php

@@ -6,7 +6,7 @@
 Route::prefix('v1')->group(function () {
     Route::controller(SiteController::class)->group(function () {
         Route::get('register', 'register');
-        Route::get('check/{hase}', 'check');
-        Route::delete('delete/{hash}', 'delete');
+        Route::get('check', 'check');
+        Route::delete('delete', 'delete');
     });
 });