fix auth again

Faithful-Resource-Pack · Oct 1, 2024 · ee7dc00 · ee7dc00
1 parent c3c43aa
commit ee7dc00
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -81,7 +81,7 @@ app.use((err: unknown, req: Request, res: Response, next: NextFunction) => {
 	if (!err) return next();
 
 	// logs and handles errors
-	const apiError = handleError(err, req.path);
+	const apiError = handleError(err, req.path, req.method);
 	if (err instanceof ValidateError) {
 		res
 			.status(422)

diff --git a/src/v2/tools/authentication.ts b/src/v2/tools/authentication.ts
@@ -89,7 +89,7 @@ export async function expressAuthentication(
 			}
 
 			// check add-on ownership
-			if (scopes.includes("addon:own")) {
+			if (scopes.includes("addon:own") || scopes.includes("addon:approved")) {
 				const idOrSlug = request.params.id_or_slug;
 				if (isSlug(idOrSlug)) {
 					const addon: Addon = (await addonService.getAddonFromSlugOrId(idOrSlug))[1];

diff --git a/src/v2/tools/handleError.ts b/src/v2/tools/handleError.ts
@@ -7,7 +7,7 @@ import { APIError } from "./errors";
  * @param err error to handle
  * @returns front-end API error to show user
  */
-export default function handleError(err: any, route: string): APIError {
+export default function handleError(err: any, route: string, method: string): APIError {
 	const code =
 		Number(
 			(typeof err.status === "number" ? err.status : err.statusCode) ||
@@ -21,7 +21,7 @@ export default function handleError(err: any, route: string): APIError {
 
 	let printed = false;
 	if (process.env.VERBOSE === "true") {
-		console.error(`[${new Date().toUTCString()}] ${route}`);
+		console.error(`[${new Date().toUTCString()}] ${method} ${route}`);
 		// if the message already includes a stack don't print it twice
 		if (message?.stack) console.error(`(${code})`, message);
 		else console.error(`(${code})`, message, "\n", stack);