操作
Bug(バグ) #1100
完了既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになる
開始日:
2010-05-25
期日:
進捗率:
100%
予定工数:
3.6 で発生するか:
Yes
3.8 で発生するか:
説明
概要¶
既にSNSにログインしているブラウザで新規登録を実行しようとすると、member/regist が404エラーになる
発生バージョン¶
OpenPNE3.5.2以降
再現手順¶
- ブラウザA でログイン
- ブラウザA でinvite@example.com 宛にSNS招待を送る
- invite@example.com に送られてきた招待URLを ブラウザA で開く
- 「このページにはアクセスできません」(404)
※ブラウザのセッションは1~4まで共有されている
改善案¶
既にログインしている状態でmember/regist にアクセスした場合、自動的にログアウトしてから表示させるようにする
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 の通り、「ログイン中であってもログイン状態が解除され、新規登録画面にアクセスできる」こと自体が適切かどうかの議論はできそうですが、現時点ではこれを仕様としておきます。
完了にします。
操作