fix: add optional chaining to queryParams access_token check

ricardogarim · ricardogarim · commit 81f1e476f083 · 2026-03-09T21:15:18.000-03:00
diff --git a/apps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.ts b/apps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.ts
@@ -75,7 +75,7 @@ API.v1.addAuthMethod((routeContext) => {
 	const authorization = routeContext.request.headers.get('authorization') ?? undefined;
 	const query = isPlainObject(routeContext.queryParams) ? routeContext.queryParams : {};
 	const accessToken = typeof query.access_token === 'string' ? query.access_token : undefined;
-	if (routeContext.queryParams.access_token) {
+	if (routeContext.queryParams?.access_token) {
 		delete routeContext.queryParams.access_token;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ API.v1.addAuthMethod((routeContext) => {`
`75`	`75`	`const authorization = routeContext.request.headers.get('authorization') ?? undefined;`
`76`	`76`	`const query = isPlainObject(routeContext.queryParams) ? routeContext.queryParams : {};`
`77`	`77`	`const accessToken = typeof query.access_token === 'string' ? query.access_token : undefined;`
`78`		`- if (routeContext.queryParams.access_token) {`
	`78`	`+ if (routeContext.queryParams?.access_token) {`
`79`	`79`	`delete routeContext.queryParams.access_token;`
`80`	`80`	`}`
`81`	`81`