Bug(バグ) #1582
Cookie が使える docomo 端末で、携帯電話個体識別番号を用いる従来式のかんたんログインができなくなってしまっている
100%
説明
#905 の調査中に発覚しました。
また、 OpenPNE 3.6beta4 および OpenPNE 3.7.0-dev では、かんたんログインの仕様変更による影響を受けて、 Cookie が使える docomo 端末で、管理画面で Cookie 内の ID によるログインをおこなわない設定がされていたとしても、従来式のかんたんログインができなくなってしまっているのを確認しました。
docomo でかんたんログインのための携帯電話個体識別番号を得るには、ログインに用いる form 要素に utn 属性を付加するか、ログインに用いる URL に guid=on パラメータを付加する必要がありますが、そのパラメータを付加するかどうかの判定に、「端末がログイン用 Cookie を所有していないかどうか」という情報を用いています。
しかし、 Cookie 内 ID を使用しない状況であっても、新規登録時には Cookie に ID を与えてますし、運用途中で Cookie 内 ID を使用する認証方法から使用しない認証方法に設定が変わった場合に、 Cookie に ID を持つ端末ではかんたんログインができないことになってしまいます。ここでの utn 等を付加するかしないかの判定は、「現在の管理画面の設定」「Cookie が利用可能な端末かどうか」の二点によっておこなうべきです。
関連するチケット
関係しているリビジョン
fixed injecting "utn" atribute and "?guid=on" parameter for mobile uid login that doesn't work under cookie-usable docomo mobile phone (fixes #1582)
removed the unused method, opWebRequest::hasMobileUidCookie() (fixes #1582)
Do you have an objection to this commit? Do you think that this is BC break? I think not.
First, this method is NOT in stable release. Next, this method is not really useful.
Last, nobody uses this method now, so it should be removed for conforming to YAGNI(http://en.wikipedia.org/wiki/You_ain't_gonna_need_it).
履歴
#1 Kousuke Ebihara が13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 cb82623f27c51cf31e5433a48eccf0231c8db699 で適用されました。
#2 Kousuke Ebihara が13年以上前に更新
- トラッカー を Bug(バグ) から Backport(バックポート) に変更
- 対象バージョン を OpenPNE 3.7.0 から OpenPNE 3.6beta5 に変更
#3 Kousuke Ebihara が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Accepted(着手) に変更
#4 Kousuke Ebihara が13年以上前に更新
- トラッカー を Backport(バックポート) から Bug(バグ) に変更
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 対象バージョン を OpenPNE 3.6beta5 から OpenPNE 3.7.0 に変更
誤ってトラッカーとステータスを変更してしまいました
#5 Kousuke Ebihara が13年以上前に更新
http://redmine.openpne.jp/issues/1583 がこの件に関する opAuthMobileUIDPlugin 側の修正です
#6 Rimpei Ogawa が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
今回の変更により hasMobileUidCookie() が使用されなくなっているので、使う予定がなければ削除してください。
#7 Kousuke Ebihara が13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 600d3d26ff35333ddce5e258bfd44901e23cf347 で適用されました。
#8 Rimpei Ogawa が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
#9 Yuma Sakata が12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKです。