プロジェクト

全般

プロフィール

Backport(バックポート) #3194

ログイン停止にしたメンバで自動ログインにチェックをいれてログインしようとしたときに「Internal Server Error」が発生する

Yuma Sakata11年以上前に追加. 11年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-10-05
期日:
進捗率:

100%


説明

概要

ログイン停止にしたメンバで自動ログインにチェックを入れてログインしようとしたときに「Internal Server Error」が発生する.

再現方法

  1. 管理画面のメンバ管理(/pc_backend.php/member/list)でIDが1のメンバのログイン停止を行う
    • より具体的には(pc_backend.php/member/reject/id/1)で「有効にする」をクリックする
  2. SNSのログイン画面を開いてIDが1のメンバのメールアドレス及びパスワードの入力と「次回から自動的にログイン」にチェックを入れて「ログイン」をクリックする
    • 「現在、サーバが混み合っているか、メンテナンス中です。」という表示がされる

原因

以下の箇所で249行目ではログインが停止されているという理由でログアウト状態となるが,251行目ではフォーム内容を確認し253行目を実行する.
この253行目を実行する際に同ファイル184行目でメンバIDを取得するが,249行目ですでにログアウト状態となっていてメンバIDを取得できないためLogicException()が発生する.

lib/user/opSecurityUser.class.php

 249     $this->initializeUserStatus();
 250
 251     if ($this->getAuthAdapter()->getAuthForm()->getValue('is_remember_me'))
 252     {
 253       $this->setRememberLoginCookie();
 254     }

 181     else
 182     {
 183       $rememberKey = opToolkit::getRandom();
 184       if (!$this->getMemberId())
 185       {
 186         throw new LogicException('No login');
 187       }
 188       $this->getMember()->setConfig('remember_key', $rememberKey);

確認環境

OpenPNE3.7.0-dev (master)
OpenPNE3.6.0-dev (stable-3.6.x)


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2469: ログイン停止にしたメンバで自動ログインにチェックをいれてログインしようとしたときに「Internal Server Error」が発生する Fixed(完了) 2011-10-05

関係しているリビジョン

リビジョン 7ce84bb0 (差分)
Yuya Watanabe11年以上前に追加

(fixes #3113, BP from #2469) fixed not to occur internal server error

履歴

#1 Yuma Sakata11年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

https://redmine.openpne.jp/issues/3113#note-4 で取り込み済みです。

#2 開 石切山11年以上前に更新

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

#3 Yuya Watanabe11年以上前に更新

note-1 の内容ですが,なぜ #3113 で対応しているのでしょうか?

#4 Yuma Sakata11年以上前に更新

note-3 指摘を回答します。

  • 経緯
    1. #3113 で #2414 の修正を取り込むためには、先に master で修正が行われていた #2469 を取り込む必要がありました
    2. なぜならば #2469#2414 とマージして行かないとコンフリクトするからです
    3. #2469 を取り込み時にBPチケットの有無を確認しなかったため、誤って #3113 に紐付けでコミットしました
    4. あとで #2469 のBPが無いことに気づいて、#3194を作成しました
  • なぜ #3113 で対応したか
    本チケットで #2469 の修正を取り込むべきですが、
    修正を取り込む前にチケットが紐付いているか確認しなかったため #3113 で対応しました

#5 Yuya Watanabe11年以上前に更新

下記コミットは #3113 と記述されていますが,本チケットが 対象チケットの BP 元の修正のため,こちらにコミットの紐付けをしなおしました.

7ce84bb02c5a37904e93af14284b2ff4ffc483bd

#6 Yuya Watanabe11年以上前に更新

  • 担当者Yuya Watanabe から Yuma Sakata に変更

#7 Yuya Watanabe11年以上前に更新

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

#8 Yuma Sakata11年以上前に更新

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

テストOKです。

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