Merged PR 502: users.repository.serviceのログ修正
## 概要 [Task2859: users.repository.serviceのログ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2859) - 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず) - 何をどう変更したか、追加したライブラリなど idTokenとversionのログ出力が正常にできるよう修正 - このPull Requestでの対象/対象外 - 影響範囲(他の機能にも影響があるか) ## レビューポイント - 特にレビューしてほしい箇所 - 軽微なものや自明なものは記載不要 - 修正範囲が大きい場合などに記載 - 全体的にや仕様を満たしているか等は本当に必要な時のみ記載 ## UIの変更 - Before/Afterのスクショなど - スクショ置き場 ## 動作確認状況 - ローカルで確認 ## 補足 - 相談、参考資料などがあれば
This commit is contained in:
parent
364bfb5135
commit
434e3d9c27
@ -343,7 +343,7 @@ export class AuthService {
|
|||||||
): Promise<boolean> {
|
): Promise<boolean> {
|
||||||
this.logger.log(
|
this.logger.log(
|
||||||
`[IN] [${context.trackingId}] ${this.isAcceptedLatestVersion.name} | params: { ` +
|
`[IN] [${context.trackingId}] ${this.isAcceptedLatestVersion.name} | params: { ` +
|
||||||
`idToken: ${idToken}, };`,
|
`idToken.sub: ${idToken.sub}, };`,
|
||||||
);
|
);
|
||||||
|
|
||||||
try {
|
try {
|
||||||
|
|||||||
@ -337,7 +337,8 @@ export class UsersRepositoryService {
|
|||||||
return await this.dataSource.transaction(async (entityManager) => {
|
return await this.dataSource.transaction(async (entityManager) => {
|
||||||
const repo = entityManager.getRepository(User);
|
const repo = entityManager.getRepository(User);
|
||||||
|
|
||||||
const accountId = (await repo.findOne({ where: { external_id } }))?.account_id;
|
const accountId = (await repo.findOne({ where: { external_id } }))
|
||||||
|
?.account_id;
|
||||||
|
|
||||||
if (!accountId) {
|
if (!accountId) {
|
||||||
throw new AccountNotFoundError('Account is Not Found.');
|
throw new AccountNotFoundError('Account is Not Found.');
|
||||||
@ -470,9 +471,9 @@ export class UsersRepositoryService {
|
|||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
if (!latestEulaInfo || !latestDpaInfo) {
|
if (!latestEulaInfo.version || !latestDpaInfo.version) {
|
||||||
throw new TermInfoNotFoundError(
|
throw new TermInfoNotFoundError(
|
||||||
`Terms info is not found. latestEulaInfo: ${latestEulaInfo}, latestDpaInfo: ${latestDpaInfo}`,
|
`Terms info is not found. latestEulaInfo: ${latestEulaInfo.version}, latestDpaInfo: ${latestDpaInfo.version}`,
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user