Skip to content

Commit d31686d

Browse files
authored
fix check and implement check/delete (#4)
* fix check and implement check/delete * cs fix
1 parent 701344e commit d31686d

File tree

2 files changed

+36
-5
lines changed

2 files changed

+36
-5
lines changed

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

Lines changed: 34 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public function register(Request $request): JsonResponse
3333
$url = $request->string('url');
3434
$key = $request->string('key');
3535

36-
if ($url->isEmpty() && $key->isEmpty()) {
36+
if ($url->isEmpty() || $key->isEmpty()) {
3737
return $this->error('BadRequest');
3838
}
3939

@@ -72,7 +72,25 @@ public function register(Request $request): JsonResponse
7272
*/
7373
public function check(Request $request): JsonResponse
7474
{
75-
return response()->json(['check']);
75+
$url = $request->string('url');
76+
$key = $request->string('key');
77+
78+
if ($url->isEmpty() || $key->isEmpty()) {
79+
return $this->error('BadRequest');
80+
}
81+
82+
$connectionService = new Connection($url, $key);
83+
84+
// Do a health check
85+
try {
86+
$connectionService->checkHealth();
87+
} catch (ServerException $e) {
88+
return $this->error($e->getMessage(), 500);
89+
} catch (ClientException|\Exception $e) {
90+
return $this->error($e->getMessage());
91+
}
92+
93+
return $this->ok();
7694
}
7795

7896
/**
@@ -82,6 +100,19 @@ public function check(Request $request): JsonResponse
82100
*/
83101
public function delete(Request $request): JsonResponse
84102
{
85-
return response()->json(['delete']);
103+
$url = $request->string('url');
104+
$key = $request->string('key');
105+
106+
if ($url->isEmpty() || $key->isEmpty()) {
107+
return $this->error('BadRequest');
108+
}
109+
110+
try {
111+
Site::where('url', $url)->where('key', $key)->delete();
112+
} catch (\Exception $e) {
113+
return $this->error($e->getMessage());
114+
}
115+
116+
return $this->ok();
86117
}
87118
}

routes/api.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
Route::prefix('v1')->group(function () {
77
Route::controller(SiteController::class)->group(function () {
88
Route::get('register', 'register');
9-
Route::get('check/{hase}', 'check');
10-
Route::delete('delete/{hash}', 'delete');
9+
Route::get('check', 'check');
10+
Route::delete('delete', 'delete');
1111
});
1212
});

0 commit comments

Comments
 (0)