fix(db-mongodb): incorrect errors logging due to invalid logic in handleError (#10575)

Previously, every error from MongoDB was logged as "Value must be
unique", as well the response code should not be `BAD_REQUEST` but
`INTERNAL_SERVER_ERROR`. `throw error` preserves the original error so
it can be traced.

Author: r1tsuu

packages/db-mongodb/src/utilities/handleError.ts

@@ -1,7 +1,6 @@
 import type { PayloadRequest } from 'payload'
 
-import httpStatus from 'http-status'
-import { APIError, ValidationError } from 'payload'
+import { ValidationError } from 'payload'
 
 export const handleError = ({
   collection,
@@ -10,18 +9,17 @@ export const handleError = ({
   req,
 }: {
   collection?: string
-  error: unknown
+  error: Error
   global?: string
   req?: Partial<PayloadRequest>
 }) => {
   if (!error || typeof error !== 'object') {
     throw error
   }
 
-  const message = req?.t ? req.t('error:valueMustBeUnique') : 'Value must be unique'
-
   // Handle uniqueness error from MongoDB
   if ('code' in error && error.code === 11000 && 'keyValue' in error && error.keyValue) {
+    const message = req?.t ? req.t('error:valueMustBeUnique') : 'Value must be unique'
     throw new ValidationError(
       {
         collection,
@@ -37,5 +35,5 @@ export const handleError = ({
     )
   }
 
-  throw new APIError(message, httpStatus.BAD_REQUEST)
+  throw error
 }