fix: remove toString() in metric attributes to avooid silly data like Object: Object

This commit is contained in:
Arvid Norlin
2024-08-26 13:30:07 +02:00
parent ca07c600d1
commit b56069e009
2 changed files with 22 additions and 22 deletions

View File

@@ -44,7 +44,7 @@ export const hotelQueryRouter = router({
console.info(
"api.hotels.hotel start",
JSON.stringify({
query: { hotelId, params: params.toString() },
query: { hotelId, params },
})
)
const apiResponse = await api.get(
@@ -74,7 +74,7 @@ export const hotelQueryRouter = router({
console.error(
"api.hotels.hotel error",
JSON.stringify({
query: { hotelId, params: params.toString() },
query: { hotelId, params },
error: {
status: apiResponse.status,
statusText: apiResponse.statusText,
@@ -99,7 +99,7 @@ export const hotelQueryRouter = router({
console.error(
"api.hotels.hotel validation error",
JSON.stringify({
query: { hotelId, params: params.toString() },
query: { hotelId, params },
error: validatedHotelData.error,
})
)
@@ -129,7 +129,7 @@ export const hotelQueryRouter = router({
console.error(
"api.hotels.hotel validation error",
JSON.stringify({
query: { hotelId, params: params.toString() },
query: { hotelId, params },
error: validatedRoom.error,
})
)
@@ -144,7 +144,7 @@ export const hotelQueryRouter = router({
console.info(
"api.hotels.hotel success",
JSON.stringify({
query: { hotelId, params: params.toString() },
query: { hotelId, params: params },
})
)
return {

View File

@@ -369,7 +369,7 @@ export const userQueryRouter = router({
const params = new URLSearchParams()
params.set("limit", "1")
getPreviousStaysCounter.add(1, { params: params.toString() })
getPreviousStaysCounter.add(1, { query: JSON.stringify({ params }) })
console.info(
"api.booking.stays.past start",
JSON.stringify({ query: { params } })
@@ -454,10 +454,10 @@ export const userQueryRouter = router({
if (cursor) {
params.offset = cursor
}
getPreviousStaysCounter.add(1, { params: params.toString() })
getPreviousStaysCounter.add(1, { query: JSON.stringify({ params }) })
console.info(
"api.booking.stays.past start",
JSON.stringify({ query: { params: params.toString() } })
JSON.stringify({ query: { params } })
)
const apiResponse = await api.get(
@@ -473,7 +473,7 @@ export const userQueryRouter = router({
if (!apiResponse.ok) {
const text = await apiResponse.text()
getPreviousStaysFailCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
error_type: "http_error",
error: JSON.stringify({
status: apiResponse.status,
@@ -484,7 +484,7 @@ export const userQueryRouter = router({
console.error(
"api.booking.stays.past error ",
JSON.stringify({
query: { params: params.toString() },
query: { params },
error: {
status: apiResponse.status,
statusText: apiResponse.statusText,
@@ -500,14 +500,14 @@ export const userQueryRouter = router({
const verifiedData = getStaysSchema.safeParse(apiJson)
if (!verifiedData.success) {
getPreviousStaysFailCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
error_type: "validation_error",
error: JSON.stringify(verifiedData.error),
})
console.error(
"api.booking.stays.past validation error ",
JSON.stringify({
query: { params: params.toString() },
query: { params },
error: verifiedData.error,
})
)
@@ -515,11 +515,11 @@ export const userQueryRouter = router({
return null
}
getPreviousStaysSuccessCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
})
console.info(
"api.booking.stays.past success",
JSON.stringify({ query: { params: params.toString() } })
JSON.stringify({ query: { params } })
)
const nextCursor =
verifiedData.data.links &&
@@ -549,11 +549,11 @@ export const userQueryRouter = router({
params.offset = cursor
}
getUpcomingStaysCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
})
console.info(
"api.booking.stays.future start",
JSON.stringify({ query: { params: params.toString() } })
JSON.stringify({ query: { params } })
)
const apiResponse = await api.get(
api.endpoints.v1.upcomingStays,
@@ -568,7 +568,7 @@ export const userQueryRouter = router({
if (!apiResponse.ok) {
const text = await apiResponse.text()
getUpcomingStaysFailCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
error_type: "http_error",
error: JSON.stringify({
status: apiResponse.status,
@@ -579,7 +579,7 @@ export const userQueryRouter = router({
console.error(
"api.booking.stays.future error ",
JSON.stringify({
query: { params: params.toString() },
query: { params },
error_type: "http_error",
error: {
status: apiResponse.status,
@@ -595,24 +595,24 @@ export const userQueryRouter = router({
const verifiedData = getStaysSchema.safeParse(apiJson)
if (!verifiedData.success) {
getUpcomingStaysFailCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
error_type: "validation_error",
error: JSON.stringify(verifiedData.error),
})
console.error(
"api.booking.stays.future validation error ",
JSON.stringify({
query: { params: params.toString() },
query: { params },
error: verifiedData.error,
})
)
return null
}
getUpcomingStaysSuccessCounter.add(1, {
query: JSON.stringify({ params: params.toString() }),
query: JSON.stringify({ params }),
})
console.info("api.booking.stays.future success", {
query: { params: params.toString() },
query: JSON.stringify({ params }),
})
const nextCursor =
verifiedData.data.links &&