Project

General

Profile

Backport(バックポート) #2139

[PATCH] Cookie for automatic login is deleted when automatic login is done (自動ログイン時に自動ログイン用のCookieが削除される)

Added by Masato Nagasawa about 8 years ago. Updated about 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-03-29
Due date:
2011-06-24
% Done:

100%


Description

Overview

Cookie for automatic login is deleted when automatic login is done.

(自動ログイン時に自動ログイン用のCookieが削除される。)

Way to repro

1. SNSに自動ログインを有効にしてログイン
2. ブラウザ終了
3. 同ブラウザでSNSにアクセス (自動ログイン・この時点でCookie消失)
4. 同ブラウザ終了
5. 同ブラウザでSNSにアクセスするとログイン画面になる。

Environment

OpenPNE3.6.x 〜

Causes

opBaseSecurityUser::isValidSiteIdentifier()

がセッション無効時に false を返す。そのため、同クラス initialize() により
ログアウト処理が実行され、自動ログイン用Cookieが削除される。

その後、自動ログインの処理が実行されるため、再現方法 3 の時点では
ログインすると考えられる。

Way to fix

opBaseSecurityUser::isValidSiteIdentifier() について セッションが無いときは
true を返すように変更


Related issues

Related to OpenPNE 3 - Bug(バグ) #1985: Cookie for automatic login is deleted when automatic login is done (自動ログイン時に自動ログイン用のCookieが削除される) Fixed(完了) 2011-03-29
Related to OpenPNE 3 - Backport(バックポート) #2453: 自動ログイン時に自動ログイン用のCookieが削除される Fixed(完了) 2011-10-03

Associated revisions

Revision 378c6316 (diff)
Added by Shogo Kawahara about 8 years ago

fixed opBaseSecurityUser because it delete Cookie for automatic log-in (fixes #2139, BP from #1985)

Revision 034d00b8 (diff)
Added by Yuya Watanabe about 8 years ago

fixed for coding standard (fixes #2139, BP from #1985)

Revision 506f56fd (diff)
Added by Shogo Kawahara about 8 years ago

fixed opBaseSecurityUser because it delete Cookie for automatic log-in (fixes #2139, BP from #1985)
(cherry picked from commit 378c6316978bcbc4e5f5bd6bf5e906f57fcd46fe)

Revision 2363b516 (diff)
Added by Yuya Watanabe about 8 years ago

fixed for coding standard (fixes #2139, BP from #1985)
(cherry picked from commit 034d00b8448679cbf152e58ebab22a3bf26dde3a)

History

#1 Updated by Kousuke Ebihara about 8 years ago

  • Due date set to 2011-06-24

#2 Updated by Kousuke Ebihara about 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee changed from Yuya Watanabe to Kousuke Ebihara

バックポート引き受けます

#3 Updated by wa ta about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 034d00b8448679cbf152e58ebab22a3bf26dde3a で適用されました。

#4 Updated by Shogo Kawahara about 8 years ago

更新履歴 378c6316978bcbc4e5f5bd6bf5e906f57fcd46fe で適用されました。

#5 Updated by isao sano about 8 years ago

テスト完了いたしました。
問題ありません。

#6 Updated by Naoya Tozuka about 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

更新箇所確認しました。レビューOKです。

#7 Updated by Mutsumi Imamura about 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

Also available in: Atom PDF