Skip to content

Commit 9fa8236

Browse files
authored
Build fixes (#500)
1 parent 8501258 commit 9fa8236

File tree

2 files changed

+18
-19
lines changed

2 files changed

+18
-19
lines changed

src/server/middleware/logs.ts

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -4,19 +4,19 @@ import { logger } from "../../utils/logger";
44
export const withRequestLogs = async (server: FastifyInstance) => {
55
server.addHook("onRequest", async (request, reply) => {
66
if (
7-
!request.routeOptions.url?.includes("static") &&
8-
!request.routeOptions.url?.includes("json") &&
7+
!request.routerPath?.includes("static") &&
8+
!request.routerPath?.includes("json") &&
99
request.method !== "OPTIONS"
1010
) {
1111
logger({
1212
service: "server",
1313
level: "info",
14-
message: `Request received - ${request.method} - ${request.routeOptions.url}`,
14+
message: `Request received - ${request.method} - ${request.routerPath}`,
1515
});
1616
}
1717

1818
if (process.env.NODE_ENV === "production") {
19-
if (request.routeOptions.url?.includes("static")) {
19+
if (request.routerPath?.includes("static")) {
2020
return reply.status(404).send({
2121
statusCode: 404,
2222
error: "Not Found",
@@ -28,16 +28,16 @@ export const withRequestLogs = async (server: FastifyInstance) => {
2828

2929
server.addHook("preHandler", async (request, reply) => {
3030
if (
31-
!request.routeOptions.url?.includes("static") &&
32-
!request.routeOptions.url?.includes("json") &&
33-
!request.routeOptions.url?.includes("/backend-wallet/import") &&
31+
!request.routerPath?.includes("static") &&
32+
!request.routerPath?.includes("json") &&
33+
!request.routerPath?.includes("/backend-wallet/import") &&
3434
request.method !== "OPTIONS"
3535
) {
3636
if (request.body && Object.keys(request.body).length > 0) {
3737
logger({
3838
service: "server",
3939
level: "info",
40-
message: `Request body - ${request.method} - ${request.routeOptions.url}`,
40+
message: `Request body - ${request.method} - ${request.routerPath}`,
4141
data: request.body,
4242
});
4343
}
@@ -55,7 +55,7 @@ export const withRequestLogs = async (server: FastifyInstance) => {
5555
logger({
5656
service: "server",
5757
level: "info",
58-
message: `Request querystring - ${request.method} - ${request.routeOptions.url}`,
58+
message: `Request querystring - ${request.method} - ${request.routerPath}`,
5959
data: request.query,
6060
});
6161
}
@@ -64,18 +64,18 @@ export const withRequestLogs = async (server: FastifyInstance) => {
6464

6565
server.addHook("onResponse", (request, reply, done) => {
6666
if (
67-
!request.routeOptions.url?.includes("static") &&
68-
!request.routeOptions.url?.includes("json") &&
67+
!request.routerPath?.includes("static") &&
68+
!request.routerPath?.includes("json") &&
6969
request.method !== "OPTIONS"
7070
) {
7171
logger({
7272
service: "server",
7373
level: "info",
7474
message: `Request completed - ${request.method} - ${
75-
reply.request.routeOptions.url
76-
} - status code: ${
77-
reply.statusCode
78-
} - Response time: ${reply.elapsedTime.toFixed(2)}ms`,
75+
reply.request.routerPath
76+
} - status code: ${reply.statusCode} - Response time: ${reply
77+
.getResponseTime()
78+
.toFixed(2)}ms`,
7979
});
8080
}
8181

src/utils/usage.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,7 @@ export const withServerUsageReporting = (server: FastifyInstance) => {
8080

8181
server.addHook("onResponse", async (request, reply) => {
8282
if (
83-
reply.request.routeOptions.url === undefined ||
84-
URLS_LIST_TO_NOT_REPORT_USAGE.has(reply.request.routeOptions.url) ||
83+
URLS_LIST_TO_NOT_REPORT_USAGE.has(reply.request.routerPath) ||
8584
reply.request.method === "OPTIONS"
8685
) {
8786
return;
@@ -101,12 +100,12 @@ export const withServerUsageReporting = (server: FastifyInstance) => {
101100
source: "engine",
102101
action: UsageEventTxActionEnum.APIRequest,
103102
clientId: thirdwebClientId,
104-
pathname: reply.request.routeOptions.url,
103+
pathname: reply.request.routerPath,
105104
chainId: chainId || undefined,
106105
walletAddress: requestParams.walletAddress || undefined,
107106
contractAddress: requestParams.contractAddress || undefined,
108107
httpStatusCode: reply.statusCode,
109-
msTotalDuration: Math.ceil(reply.elapsedTime),
108+
msTotalDuration: Math.ceil(reply.getResponseTime()),
110109
};
111110

112111
fetch(env.CLIENT_ANALYTICS_URL, {

0 commit comments

Comments
 (0)