プロジェクト

全般

プロフィール

Backport(バックポート) #3194

完了

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

Yuma Sakata さんが約12年前に追加. 約12年前に更新.

ステータス:
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)


関連するチケット 1 (0件未完了1件完了)

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

操作

Yuma Sakata さんが約12年前に更新

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

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

開 石切山 さんが約12年前に更新

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

Yuya Watanabe さんが約12年前に更新

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

Yuma Sakata さんが約12年前に更新

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

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

Yuya Watanabe さんが約12年前に更新

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

7ce84bb02c5a37904e93af14284b2ff4ffc483bd

Yuya Watanabe さんが約12年前に更新

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

Yuya Watanabe さんが約12年前に更新

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

Yuma Sakata さんが約12年前に更新

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

テストOKです。

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