fix: dont forward internal exceptions

Signed-off-by: Daniel Kesselberg <[email protected]>
nextcloud · Oct 30, 2023 · 261c9bd · 261c9bd
1 parent 8f39923
commit 261c9bd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/lib/Controller/AppointmentConfigController.php b/lib/Controller/AppointmentConfigController.php
@@ -199,7 +199,7 @@ public function create(
 			return JsonResponse::success($appointmentConfig);
 		} catch (ServiceException $e) {
 			$this->logger->error('Could not create new configuration', ['exception' => $e]);
-			return JsonResponse::errorFromThrowable($e);
+			return JsonResponse::fail($e->getMessage(), Http::STATUS_UNPROCESSABLE_ENTITY);
 		}
 	}
 
@@ -257,7 +257,7 @@ public function update(
 			$appointmentConfig = $this->appointmentConfigService->findByIdAndUser($id, $this->userId);
 		} catch (ClientException $e) {
 			$this->logger->error('Could not find configuration with id ' . $id, ['exception' => $e]);
-			return JsonResponse::errorFromThrowable($e);
+			return JsonResponse::fail($e->getMessage(), Http::STATUS_NOT_FOUND);
 		}
 
 		$appointmentConfig->setName($name);
@@ -284,7 +284,7 @@ public function update(
 			return JsonResponse::success($appointmentConfig);
 		} catch (ServiceException $e) {
 			$this->logger->error('Could not update configuration with id ' . $id, ['exception' => $e]);
-			return JsonResponse::errorFromThrowable($e, 403);
+			return JsonResponse::fail($e->getMessage(), Http::STATUS_UNPROCESSABLE_ENTITY);
 		}
 	}
 
@@ -303,7 +303,7 @@ public function destroy(int $id): JsonResponse {
 			return JsonResponse::success();
 		} catch (ServiceException $e) {
 			$this->logger->error('Could not delete configuration with id ' . $id, ['exception' => $e]);
-			return JsonResponse::errorFromThrowable($e, 403);
+			return JsonResponse::fail($e->getMessage(), Http::STATUS_NOT_FOUND);
 		}
 	}
 }
diff --git a/src/services/appointmentConfigService.js b/src/services/appointmentConfigService.js
@@ -32,6 +32,7 @@ import logger from '../utils/logger.js'
  * @return {Promise<AppointmentConfig>} Full appointment config with an id
  */
 export async function createConfig(config) {
+	logger.debug('Creating appointment config', { config })
 	const url = generateUrl('/apps/calendar/v1/appointment_configs')
 
 	const response = await axios.post(url, config)
@@ -45,6 +46,7 @@ export async function createConfig(config) {
  * @return {Promise<void>}
  */
 export async function deleteConfig(id) {
+	logger.debug('Deleting appointment config', { config })
 	const url = generateUrl('/apps/calendar/v1/appointment_configs/{id}', {
 		id,
 	})
@@ -59,7 +61,7 @@ export async function deleteConfig(id) {
  * @return {Promise<AppointmentConfig>} Updated appointment config as stored in the backend
  */
 export async function updateConfig(config) {
-	logger.info('Deleting config', { config, id: config.id })
+	logger.debug('Updating appointment config', { config })
 	const url = generateUrl('/apps/calendar/v1/appointment_configs/{id}', {
 		id: config.id,
 	})