プロジェクト

全般

プロフィール

Bug(バグ) #1836

管理画面でログイン停止にしてもログイン可能

Yuma Sakata13年以上前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2010-12-07
期日:
進捗率:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

説明

現象

管理画面でログイン停止にしてもPCでログイン可能。
バージョン3.4と3.2では再現できませんでした。

※今回、URLは省略させていただきます。

OpenPNEのバージョン 3.6beta7-dev

再現手順

  1. 管理画面でログイン停止状態にして有効にする。
  2. ログイン停止状態のIDでログインする。
  3. ログイン停止状態のはずなのに、ログイン可能。

原因

修正内容


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #1880: 管理画面でログイン停止にしてもログイン可能 Fixed(完了) 2010-12-07 2011-06-24
関連している OpenPNE 3 - Bug(バグ) #1182: 携帯でSNS強制退会直後にアクセスするとCredentials Required画面が表示される Fixed(完了) 2010-06-18

関係しているリビジョン

リビジョン 04558ea8 (差分)
Masato Nagasawa13年以上前に追加

fixed login reject (fixes #1836)

履歴

#1 Kousuke Ebihara13年以上前に更新

  • 3.6 で発生するかYes にセット

#2 Kousuke Ebihara13年以上前に更新

  • 優先度Normal(通常) から High(高め) に変更

#3 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.7.0 にセット

#4 Masato Nagasawa13年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Masato Nagasawa にセット

#5 Masato Nagasawa13年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 04558ea88ff2763a362ebd179b0bd956be468218 で適用されました。

#6 Masato Nagasawa13年以上前に更新

以下のテストを行っています。

ログイン状態 ログイン停止対象 ログイン対象 期待結果
未ログイン メンバーA メンバーA ログインできない
未ログイン メンバーB メンバーB ログインできない
未ログイン メンバーA メンバーB ログインできる
未ログイン メンバーB メンバーA ログインできる
メンバーA メンバーA メンバーA ログアウトされる
メンバーB メンバーB メンバーB ログアウトされる
メンバーB メンバーA メンバーB ログイン継続
メンバーA メンバーB メンバーA ログイン継続

#7 Masato Nagasawa13年以上前に更新

isActive() が false であった場合に、
setAuthenticated() の値が false になるような変更を行ったので、
影響のありそうな以下の動作テストも行いました。

  • 友達招待で登録からログインまでできるか
  • 新規登録で登録からログインまでできるか

#8 Naoya Tozukaほぼ13年前に更新

変更点確認しました。OKです。

メモ

私見ですが、opSecurityUser クラスにおける "isSNSMember" という名称の使われ方が曖昧なので、

  • isSNSMember() → isValidSNSMember()
  • setIsSNSMember(bool) → setSNSMemberCredential(bool)

のように改名したらよいのではないかと思いました。(tozuka)

#9 Naoya Tozukaほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#10 Yuma Sakata12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

#11 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

他の形式にエクスポート: Atom PDF