Skip to content

Commit 5f41acf

Browse files
gijs-blankengijs-blanken
authored
add #report when handling exceptions in the api (#21023)
1 parent b2abf36 commit 5f41acf

File tree

8 files changed

+47
-0
lines changed

8 files changed

+47
-0
lines changed

Diff for: ‎modules/openapi-generator/src/main/resources/php-laravel/api_controller.mustache

+1
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ class {{controllerName}} extends Controller
9191
$apiResult = $this->api->{{operationId}}({{#allParams}}${{paramName}}{{^-last}}, {{/-last}}{{/allParams}});
9292
} catch (\Exception $exception) {
9393
// This shouldn't happen
94+
report($exception);
9495
return response()->json(['error' => $exception->getMessage()], 500);
9596
}
9697

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/AnotherFakeController.php

+1
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ public function call123TestSpecialTags(Request $request): JsonResponse
7070
$apiResult = $this->api->call123TestSpecialTags($client);
7171
} catch (\Exception $exception) {
7272
// This shouldn't happen
73+
report($exception);
7374
return response()->json(['error' => $exception->getMessage()], 500);
7475
}
7576

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/DefaultController.php

+1
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ public function fooGet(Request $request): JsonResponse
6868
$apiResult = $this->api->fooGet();
6969
} catch (\Exception $exception) {
7070
// This shouldn't happen
71+
report($exception);
7172
return response()->json(['error' => $exception->getMessage()], 500);
7273
}
7374

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/FakeClassnameTags123Controller.php

+1
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ public function testClassname(Request $request): JsonResponse
7070
$apiResult = $this->api->testClassname($client);
7171
} catch (\Exception $exception) {
7272
// This shouldn't happen
73+
report($exception);
7374
return response()->json(['error' => $exception->getMessage()], 500);
7475
}
7576

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/FakeController.php

+22
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ public function fakeBigDecimalMap(Request $request): JsonResponse
6868
$apiResult = $this->api->fakeBigDecimalMap();
6969
} catch (\Exception $exception) {
7070
// This shouldn't happen
71+
report($exception);
7172
return response()->json(['error' => $exception->getMessage()], 500);
7273
}
7374

@@ -106,6 +107,7 @@ public function fakeHealthGet(Request $request): JsonResponse
106107
$apiResult = $this->api->fakeHealthGet();
107108
} catch (\Exception $exception) {
108109
// This shouldn't happen
110+
report($exception);
109111
return response()->json(['error' => $exception->getMessage()], 500);
110112
}
111113

@@ -150,6 +152,7 @@ public function fakeHttpSignatureTest(Request $request): JsonResponse
150152
$apiResult = $this->api->fakeHttpSignatureTest($pet, $query1, $header1);
151153
} catch (\Exception $exception) {
152154
// This shouldn't happen
155+
report($exception);
153156
return response()->json(['error' => $exception->getMessage()], 500);
154157
}
155158

@@ -190,6 +193,7 @@ public function fakeOuterBooleanSerialize(Request $request): JsonResponse
190193
$apiResult = $this->api->fakeOuterBooleanSerialize($body);
191194
} catch (\Exception $exception) {
192195
// This shouldn't happen
196+
report($exception);
193197
return response()->json(['error' => $exception->getMessage()], 500);
194198
}
195199

@@ -230,6 +234,7 @@ public function fakeOuterCompositeSerialize(Request $request): JsonResponse
230234
$apiResult = $this->api->fakeOuterCompositeSerialize($outerComposite);
231235
} catch (\Exception $exception) {
232236
// This shouldn't happen
237+
report($exception);
233238
return response()->json(['error' => $exception->getMessage()], 500);
234239
}
235240

@@ -270,6 +275,7 @@ public function fakeOuterNumberSerialize(Request $request): JsonResponse
270275
$apiResult = $this->api->fakeOuterNumberSerialize($body);
271276
} catch (\Exception $exception) {
272277
// This shouldn't happen
278+
report($exception);
273279
return response()->json(['error' => $exception->getMessage()], 500);
274280
}
275281

@@ -310,6 +316,7 @@ public function fakeOuterStringSerialize(Request $request): JsonResponse
310316
$apiResult = $this->api->fakeOuterStringSerialize($body);
311317
} catch (\Exception $exception) {
312318
// This shouldn't happen
319+
report($exception);
313320
return response()->json(['error' => $exception->getMessage()], 500);
314321
}
315322

@@ -350,6 +357,7 @@ public function fakePropertyEnumIntegerSerialize(Request $request): JsonResponse
350357
$apiResult = $this->api->fakePropertyEnumIntegerSerialize($outerObjectWithEnumProperty);
351358
} catch (\Exception $exception) {
352359
// This shouldn't happen
360+
report($exception);
353361
return response()->json(['error' => $exception->getMessage()], 500);
354362
}
355363

@@ -390,6 +398,7 @@ public function testAdditionalPropertiesReference(Request $request): JsonRespons
390398
$apiResult = $this->api->testAdditionalPropertiesReference($requestBody);
391399
} catch (\Exception $exception) {
392400
// This shouldn't happen
401+
report($exception);
393402
return response()->json(['error' => $exception->getMessage()], 500);
394403
}
395404

@@ -430,6 +439,7 @@ public function testBodyWithBinary(Request $request): JsonResponse
430439
$apiResult = $this->api->testBodyWithBinary($body);
431440
} catch (\Exception $exception) {
432441
// This shouldn't happen
442+
report($exception);
433443
return response()->json(['error' => $exception->getMessage()], 500);
434444
}
435445

@@ -470,6 +480,7 @@ public function testBodyWithFileSchema(Request $request): JsonResponse
470480
$apiResult = $this->api->testBodyWithFileSchema($fileSchemaTestClass);
471481
} catch (\Exception $exception) {
472482
// This shouldn't happen
483+
report($exception);
473484
return response()->json(['error' => $exception->getMessage()], 500);
474485
}
475486

@@ -512,6 +523,7 @@ public function testBodyWithQueryParams(Request $request): JsonResponse
512523
$apiResult = $this->api->testBodyWithQueryParams($query, $user);
513524
} catch (\Exception $exception) {
514525
// This shouldn't happen
526+
report($exception);
515527
return response()->json(['error' => $exception->getMessage()], 500);
516528
}
517529

@@ -552,6 +564,7 @@ public function testClientModel(Request $request): JsonResponse
552564
$apiResult = $this->api->testClientModel($client);
553565
} catch (\Exception $exception) {
554566
// This shouldn't happen
567+
report($exception);
555568
return response()->json(['error' => $exception->getMessage()], 500);
556569
}
557570

@@ -672,6 +685,7 @@ public function testEndpointParameters(Request $request): JsonResponse
672685
$apiResult = $this->api->testEndpointParameters($number, $double, $patternWithoutDelimiter, $byte, $integer, $int32, $int64, $float, $string, $binary, $date, $dateTime, $password, $callback);
673686
} catch (\Exception $exception) {
674687
// This shouldn't happen
688+
report($exception);
675689
return response()->json(['error' => $exception->getMessage()], 500);
676690
}
677691

@@ -754,6 +768,7 @@ public function testEnumParameters(Request $request): JsonResponse
754768
$apiResult = $this->api->testEnumParameters($enumHeaderStringArray, $enumHeaderString, $enumQueryStringArray, $enumQueryString, $enumQueryInteger, $enumQueryDouble, $enumQueryModelArray, $enumFormStringArray, $enumFormString);
755769
} catch (\Exception $exception) {
756770
// This shouldn't happen
771+
report($exception);
757772
return response()->json(['error' => $exception->getMessage()], 500);
758773
}
759774

@@ -829,6 +844,7 @@ public function testGroupParameters(Request $request): JsonResponse
829844
$apiResult = $this->api->testGroupParameters($requiredStringGroup, $requiredBooleanGroup, $requiredInt64Group, $stringGroup, $booleanGroup, $int64Group);
830845
} catch (\Exception $exception) {
831846
// This shouldn't happen
847+
report($exception);
832848
return response()->json(['error' => $exception->getMessage()], 500);
833849
}
834850

@@ -869,6 +885,7 @@ public function testInlineAdditionalProperties(Request $request): JsonResponse
869885
$apiResult = $this->api->testInlineAdditionalProperties($requestBody);
870886
} catch (\Exception $exception) {
871887
// This shouldn't happen
888+
report($exception);
872889
return response()->json(['error' => $exception->getMessage()], 500);
873890
}
874891

@@ -909,6 +926,7 @@ public function testInlineFreeformAdditionalProperties(Request $request): JsonRe
909926
$apiResult = $this->api->testInlineFreeformAdditionalProperties($testInlineFreeformAdditionalPropertiesRequest);
910927
} catch (\Exception $exception) {
911928
// This shouldn't happen
929+
report($exception);
912930
return response()->json(['error' => $exception->getMessage()], 500);
913931
}
914932

@@ -959,6 +977,7 @@ public function testJsonFormData(Request $request): JsonResponse
959977
$apiResult = $this->api->testJsonFormData($param, $param2);
960978
} catch (\Exception $exception) {
961979
// This shouldn't happen
980+
report($exception);
962981
return response()->json(['error' => $exception->getMessage()], 500);
963982
}
964983

@@ -999,6 +1018,7 @@ public function testNullable(Request $request): JsonResponse
9991018
$apiResult = $this->api->testNullable($childWithNullable);
10001019
} catch (\Exception $exception) {
10011020
// This shouldn't happen
1021+
report($exception);
10021022
return response()->json(['error' => $exception->getMessage()], 500);
10031023
}
10041024

@@ -1077,6 +1097,7 @@ public function testQueryParameterCollectionFormat(Request $request): JsonRespon
10771097
$apiResult = $this->api->testQueryParameterCollectionFormat($pipe, $ioutil, $http, $url, $context, $allowEmpty, $language);
10781098
} catch (\Exception $exception) {
10791099
// This shouldn't happen
1100+
report($exception);
10801101
return response()->json(['error' => $exception->getMessage()], 500);
10811102
}
10821103

@@ -1117,6 +1138,7 @@ public function testStringMapReference(Request $request): JsonResponse
11171138
$apiResult = $this->api->testStringMapReference($requestBody);
11181139
} catch (\Exception $exception) {
11191140
// This shouldn't happen
1141+
report($exception);
11201142
return response()->json(['error' => $exception->getMessage()], 500);
11211143
}
11221144

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/PetController.php

+9
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ public function addPet(Request $request): JsonResponse
7070
$apiResult = $this->api->addPet($pet);
7171
} catch (\Exception $exception) {
7272
// This shouldn't happen
73+
report($exception);
7374
return response()->json(['error' => $exception->getMessage()], 500);
7475
}
7576

@@ -122,6 +123,7 @@ public function deletePet(Request $request, int $petId): JsonResponse
122123
$apiResult = $this->api->deletePet($petId, $apiKey);
123124
} catch (\Exception $exception) {
124125
// This shouldn't happen
126+
report($exception);
125127
return response()->json(['error' => $exception->getMessage()], 500);
126128
}
127129

@@ -170,6 +172,7 @@ public function findPetsByStatus(Request $request): JsonResponse
170172
$apiResult = $this->api->findPetsByStatus($status);
171173
} catch (\Exception $exception) {
172174
// This shouldn't happen
175+
report($exception);
173176
return response()->json(['error' => $exception->getMessage()], 500);
174177
}
175178

@@ -220,6 +223,7 @@ public function findPetsByTags(Request $request): JsonResponse
220223
$apiResult = $this->api->findPetsByTags($tags);
221224
} catch (\Exception $exception) {
222225
// This shouldn't happen
226+
report($exception);
223227
return response()->json(['error' => $exception->getMessage()], 500);
224228
}
225229

@@ -268,6 +272,7 @@ public function getPetById(Request $request, int $petId): JsonResponse
268272
$apiResult = $this->api->getPetById($petId);
269273
} catch (\Exception $exception) {
270274
// This shouldn't happen
275+
report($exception);
271276
return response()->json(['error' => $exception->getMessage()], 500);
272277
}
273278

@@ -316,6 +321,7 @@ public function updatePet(Request $request): JsonResponse
316321
$apiResult = $this->api->updatePet($pet);
317322
} catch (\Exception $exception) {
318323
// This shouldn't happen
324+
report($exception);
319325
return response()->json(['error' => $exception->getMessage()], 500);
320326
}
321327

@@ -381,6 +387,7 @@ public function updatePetWithForm(Request $request, int $petId): JsonResponse
381387
$apiResult = $this->api->updatePetWithForm($petId, $name, $status);
382388
} catch (\Exception $exception) {
383389
// This shouldn't happen
390+
report($exception);
384391
return response()->json(['error' => $exception->getMessage()], 500);
385392
}
386393

@@ -438,6 +445,7 @@ public function uploadFile(Request $request, int $petId): JsonResponse
438445
$apiResult = $this->api->uploadFile($petId, $additionalMetadata, $file);
439446
} catch (\Exception $exception) {
440447
// This shouldn't happen
448+
report($exception);
441449
return response()->json(['error' => $exception->getMessage()], 500);
442450
}
443451

@@ -492,6 +500,7 @@ public function uploadFileWithRequiredFile(Request $request, int $petId): JsonRe
492500
$apiResult = $this->api->uploadFileWithRequiredFile($petId, $requiredFile, $additionalMetadata);
493501
} catch (\Exception $exception) {
494502
// This shouldn't happen
503+
report($exception);
495504
return response()->json(['error' => $exception->getMessage()], 500);
496505
}
497506

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/StoreController.php

+4
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,7 @@ public function deleteOrder(Request $request, string $orderId): JsonResponse
7373
$apiResult = $this->api->deleteOrder($orderId);
7474
} catch (\Exception $exception) {
7575
// This shouldn't happen
76+
report($exception);
7677
return response()->json(['error' => $exception->getMessage()], 500);
7778
}
7879

@@ -115,6 +116,7 @@ public function getInventory(Request $request): JsonResponse
115116
$apiResult = $this->api->getInventory();
116117
} catch (\Exception $exception) {
117118
// This shouldn't happen
119+
report($exception);
118120
return response()->json(['error' => $exception->getMessage()], 500);
119121
}
120122

@@ -161,6 +163,7 @@ public function getOrderById(Request $request, int $orderId): JsonResponse
161163
$apiResult = $this->api->getOrderById($orderId);
162164
} catch (\Exception $exception) {
163165
// This shouldn't happen
166+
report($exception);
164167
return response()->json(['error' => $exception->getMessage()], 500);
165168
}
166169

@@ -209,6 +212,7 @@ public function placeOrder(Request $request): JsonResponse
209212
$apiResult = $this->api->placeOrder($order);
210213
} catch (\Exception $exception) {
211214
// This shouldn't happen
215+
report($exception);
212216
return response()->json(['error' => $exception->getMessage()], 500);
213217
}
214218

Diff for: ‎samples/server/petstore/php-laravel/Http/Controllers/UserController.php

+8
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ public function createUser(Request $request): JsonResponse
7070
$apiResult = $this->api->createUser($user);
7171
} catch (\Exception $exception) {
7272
// This shouldn't happen
73+
report($exception);
7374
return response()->json(['error' => $exception->getMessage()], 500);
7475
}
7576

@@ -110,6 +111,7 @@ public function createUsersWithArrayInput(Request $request): JsonResponse
110111
$apiResult = $this->api->createUsersWithArrayInput($user);
111112
} catch (\Exception $exception) {
112113
// This shouldn't happen
114+
report($exception);
113115
return response()->json(['error' => $exception->getMessage()], 500);
114116
}
115117

@@ -150,6 +152,7 @@ public function createUsersWithListInput(Request $request): JsonResponse
150152
$apiResult = $this->api->createUsersWithListInput($user);
151153
} catch (\Exception $exception) {
152154
// This shouldn't happen
155+
report($exception);
153156
return response()->json(['error' => $exception->getMessage()], 500);
154157
}
155158

@@ -193,6 +196,7 @@ public function deleteUser(Request $request, string $username): JsonResponse
193196
$apiResult = $this->api->deleteUser($username);
194197
} catch (\Exception $exception) {
195198
// This shouldn't happen
199+
report($exception);
196200
return response()->json(['error' => $exception->getMessage()], 500);
197201
}
198202

@@ -240,6 +244,7 @@ public function getUserByName(Request $request, string $username): JsonResponse
240244
$apiResult = $this->api->getUserByName($username);
241245
} catch (\Exception $exception) {
242246
// This shouldn't happen
247+
report($exception);
243248
return response()->json(['error' => $exception->getMessage()], 500);
244249
}
245250

@@ -298,6 +303,7 @@ public function loginUser(Request $request): JsonResponse
298303
$apiResult = $this->api->loginUser($username, $password);
299304
} catch (\Exception $exception) {
300305
// This shouldn't happen
306+
report($exception);
301307
return response()->json(['error' => $exception->getMessage()], 500);
302308
}
303309

@@ -340,6 +346,7 @@ public function logoutUser(Request $request): JsonResponse
340346
$apiResult = $this->api->logoutUser();
341347
} catch (\Exception $exception) {
342348
// This shouldn't happen
349+
report($exception);
343350
return response()->json(['error' => $exception->getMessage()], 500);
344351
}
345352

@@ -381,6 +388,7 @@ public function updateUser(Request $request, string $username): JsonResponse
381388
$apiResult = $this->api->updateUser($username, $user);
382389
} catch (\Exception $exception) {
383390
// This shouldn't happen
391+
report($exception);
384392
return response()->json(['error' => $exception->getMessage()], 500);
385393
}
386394

0 commit comments

Comments
 (0)