1
0
forked from mirror/misskey

fix(server): エラーのスタックトレースは返さないように

Fix #10064
This commit is contained in:
syuilo 2023-02-26 17:12:15 +09:00
parent cedfb85b60
commit cc149e2f46
2 changed files with 6 additions and 2 deletions

View File

@ -27,6 +27,7 @@ You should also include the user name that made the change.
- fix(client): Android ChromeでPWAとしてインストールできない問題を修正 - fix(client): Android ChromeでPWAとしてインストールできない問題を修正
- 未知のユーザーが deleteActor されたら処理をスキップする - 未知のユーザーが deleteActor されたら処理をスキップする
- fix(server): notes/createで、fileIdsと見つかったファイルの数が異なる場合はエラーにする - fix(server): notes/createで、fileIdsと見つかったファイルの数が異なる場合はエラーにする
- fix(server): エラーのスタックトレースは返さないように
## 13.7.5 (2023/02/24) ## 13.7.5 (2023/02/24)

View File

@ -2,6 +2,7 @@ import { pipeline } from 'node:stream';
import * as fs from 'node:fs'; import * as fs from 'node:fs';
import { promisify } from 'node:util'; import { promisify } from 'node:util';
import { Inject, Injectable } from '@nestjs/common'; import { Inject, Injectable } from '@nestjs/common';
import { v4 as uuid } from 'uuid';
import { DI } from '@/di-symbols.js'; import { DI } from '@/di-symbols.js';
import { getIpHash } from '@/misc/get-ip-hash.js'; import { getIpHash } from '@/misc/get-ip-hash.js';
import type { LocalUser, User } from '@/models/entities/User.js'; import type { LocalUser, User } from '@/models/entities/User.js';
@ -320,6 +321,7 @@ export class ApiCallService implements OnApplicationShutdown {
if (err instanceof ApiError) { if (err instanceof ApiError) {
throw err; throw err;
} else { } else {
const errId = uuid();
this.logger.error(`Internal error occurred in ${ep.name}: ${err.message}`, { this.logger.error(`Internal error occurred in ${ep.name}: ${err.message}`, {
ep: ep.name, ep: ep.name,
ps: data, ps: data,
@ -327,14 +329,15 @@ export class ApiCallService implements OnApplicationShutdown {
message: err.message, message: err.message,
code: err.name, code: err.name,
stack: err.stack, stack: err.stack,
id: errId,
}, },
}); });
console.error(err); console.error(err, errId);
throw new ApiError(null, { throw new ApiError(null, {
e: { e: {
message: err.message, message: err.message,
code: err.name, code: err.name,
stack: err.stack, id: errId,
}, },
}); });
} }