プロジェクト

全般

プロフィール

Bug(バグ) #1100

完了

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

Kiwa Sakai さんが14年以上前に追加. 13年以上前に更新.

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

100%

予定工数:
3.6 で発生するか:
Yes
3.8 で発生するか:

説明

概要

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

発生バージョン

OpenPNE3.5.2以降

再現手順

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

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

改善案

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


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

関連している OpenPNE 3 - Backport(バックポート) #2150: 既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになるFixed(完了)Maki Takahashi2011-06-09

操作
関連している OpenPNE 3 - Bug(バグ) #1031: 新規登録画面が404エラーになる場合があるNew(新規)2010-05-10

操作

Rimpei Ogawa さんが約14年前に更新

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

Kousuke Ebihara さんが約14年前に更新

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

Kousuke Ebihara さんが約14年前に更新

  • 対象バージョンOpenPNE 3.6beta6 から OpenPNE 3.7.0 に変更

Masato Nagasawa さんが13年以上前に更新

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

Masato Nagasawa さんが13年以上前に更新

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

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

Naoya Tozuka さんが13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 担当者 を削除 (Masato Nagasawa)
  • 進捗率50 から 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を長澤くん実装で通してからでも構わないと思っている程度

Minoru Takai さんが13年以上前に更新

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

動作テスト

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

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

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

完了にします。

Minoru Takai さんが13年以上前に更新

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

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

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