Project

General

Profile

Bug(バグ) #1100

既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになる

Added by Kiwa Sakai about 9 years ago. Updated almost 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
-
Target version:
Start date:
2010-05-25
Due date:
% Done:

100%

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

Description

概要

既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになる

発生バージョン

OpenPNE3.5.2以降

再現手順

  1. ブラウザA でログイン
  2. ブラウザA で 宛にSNS招待を送る
  3. に送られてきた招待URLを ブラウザA で開く
  4. 「このページにはアクセスできません」(404)

※ブラウザのセッションは1~4まで共有されている

改善案

既にログインしている状態でmember/regist にアクセスした場合、自動的にログアウトしてから表示させるようにする


Related issues

Related to OpenPNE 3 - Backport(バックポート) #2150: 既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになる Fixed(完了) 2011-06-09
Related to OpenPNE 3 - Bug(バグ) #1031: 新規登録画面が404エラーになる場合がある New(新規) 2010-05-10

Associated revisions

Revision e5c8e314 (diff)
Added by Masato Nagasawa about 8 years ago

fixed clear the session data, if to access the "member/regis" (fixes #1100)

History

#1 Updated by Rimpei Ogawa almost 9 years ago

  • 3.6 で発生するか set to Yes

#2 Updated by Kousuke Ebihara over 8 years ago

  • Target version set to OpenPNE 3.6beta6

#3 Updated by Kousuke Ebihara over 8 years ago

  • Target version changed from OpenPNE 3.6beta6 to OpenPNE 3.7.0

#5 Updated by Masato Nagasawa about 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Masato Nagasawa

#6 Updated by Masato Nagasawa about 8 years ago

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

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

#7 Updated by Naoya Tozuka almost 8 years ago

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

修正箇所確認しました。OKです。

メモ

本チケットの改善案仕様(新規登録ページにアクセスした際にログイン中のユーザが自動ログアウトする)については、議論の余地があるのではないかと思われます。

チャットログ:

tozuka> Bug#1100 の「既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになる」の解決策は、本当に「既にログインしているセッションを自動的にログアウトして、新規登録を実行する」でよいのかちょっと悩むけどこれはそういう話でいいのかな http://redmine.openpne.jp/issues/1100
takai> @tozuka #1100
 1. member_id=Xでログインする
 2. 招待メールを貰うか何かする
 3. 招待メールから新規登録ページに飛ぶ
 4. member_id=Xをログアウトする
 5. 新規登録ページに遷移させる
 この流れ自体が適切じゃないんじゃないか、という悩みですか?(それとも別なのかな)
tozuka> @takai 個人的には、この流れではユーザにログアウトを促す方がいいのかなと思っていたという程度です
takai> ログイン中に新規登録ページにアクセスすると何が起こって欲しいのかっていうところですよね。
 // 個人的には「新規登録ページにアクセスした瞬間に自動ログアウト」でよいんじゃないかと思っている
tozuka> @takai 自分の使い方としてはそれ(「新規登録ページにアクセスした瞬間に自動ログアウト」)で構わないというかその方が便利なんだけど、なんか複垢作成推奨っぽく感じていて。
tozuka> (といっても、議論は一旦#1100を長澤くん実装で通してからでも構わないと思っている程度)
takai> なるほど、確かに若干は議論の余地がありそうですね(画面遷移とか)
takahashi> (#1100がどうなるか気になってます・・・BPの担当になったもので・・・)
tozuka> チケットのDescriptionにある改善案を覆すところまではしていない
tozuka> 404エラーが出ている現状の方が問題だと思うので取り急ぎ修正したいという気持ち
tozuka> でも議論の余地はあると思う旨チケットにメモ
tozuka> そしてテストフェーズへ
takai> // 議論の余地はありそうだけど、議論は一旦#1100を長澤くん実装で通してからでも構わないと思っている程度

#8 Updated by Minoru Takai almost 8 years ago

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

動作テスト

修正前のコードで「ログイン中に新規登録画面にアクセス」を行うとエラーページとなることを確認し、修正後のコードではログイン中であってもログイン状態が解除され、新規登録画面にアクセスできることを確認しました。

また修正内容を見る限り、この修正による副作用もないと判断しています。この問題を解消(※)するための修正として妥当であると判断し、動作テストをOKとします。

※ note-7 の通り、「ログイン中であってもログイン状態が解除され、新規登録画面にアクセスできる」こと自体が適切かどうかの議論はできそうですが、現時点ではこれを仕様としておきます。

完了にします。

#9 Updated by Minoru Takai almost 8 years ago

  • Priority changed from Normal(通常) to High(高め)

優先度を 3.6.x 向けチケットと合わせます。

Also available in: Atom PDF