EC-CUBE / EC-CUBE/ec-cube

LoginHistoryStatusが取得できない場合に、ログイン履歴が保存されない

Open
#5,663 1 comment 0 reactions 0 assignees View on GitHub
bug:Low bugbounty2022:issue:entry bugbounty2022:issue:recognition bugbounty2022:issue:runbook
Dominant language
PHP
Stars
788
Forks
719
Avg merge
4d 4h
Merged PRs (30d)
39

Description

**※あまり想定されないケースだなと思いつつIssue立ててます**

## 概要(Overview)
ログイン成功/失敗時にログイン履歴を記録する機能がありますが、LoginHistoryStatusを取得できなかった場合には記録されないロジックになっていました。

https://github.com/EC-CUBE/ec-cube/blob/d5b681c4fbef38c313454110ddcb940c741efab2/src/Eccube/EventListener/LoginHistoryListener.php#L88-L91
https://github.com/EC-CUBE/ec-cube/blob/d5b681c4fbef38c313454110ddcb940c741efab2/src/Eccube/EventListener/LoginHistoryListener.php#L114-L117

## 期待する内容(Expect) or 要望 (Requirement)
ログイン履歴というセキュリティに関する履歴の性質上、LoginHistoryStatusが正常に取得できなかったとしてもログイン履歴は残るべきではないかと考えます。

## 再現手順(Procedure)
1. マスターデータ管理で `mtb_login_history_status` を選択し、(0, 失敗) を (100, 失敗) に書き換える。
2. 別ブラウザから管理画面へのログインをわざと "失敗" する。
3. マスターデータ管理で `mtb_login_history_status` を選択し、(100, 失敗) を (0, 失敗) に戻す。※これ戻さないとログイン履歴画面が表示できないので
4. ログイン履歴を確認する。→手順(2)での失敗が記録されていない(NG)

### 環境 (environment)
+ EC-CUBE: 4.2-beta2

## 関連情報 (Ref)
なし

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.