プロジェクト

全般

プロフィール

Bug(バグ) #1582

Cookie が使える docomo 端末で、携帯電話個体識別番号を用いる従来式のかんたんログインができなくなってしまっている

Kousuke Ebihara13年以上前に追加. 8年以上前に更新.

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

100%

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

説明

#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 が利用可能な端末かどうか」の二点によっておこなうべきです。


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #1587: Cookie が使える docomo 端末で、携帯電話個体識別番号を用いる従来式のかんたんログインができなくなってしまっている Fixed(完了) 2010-09-13
関連している opAuthMobileUIDPlugin - Enhancement(機能追加・改善) #1583: opAuthLoginFormMobileUID::isUtn() の返り値を、管理画面の設定値によって決定するようにする Fixed(完了) 2010-09-13

関係しているリビジョン

リビジョン cb82623f (差分)
Kousuke Ebihara13年以上前に追加

fixed injecting "utn" atribute and "?guid=on" parameter for mobile uid login that doesn't work under cookie-usable docomo mobile phone (fixes #1582)

リビジョン 600d3d26 (差分)
Kousuke Ebihara13年以上前に追加

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 Ebihara13年以上前に更新

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

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

#2 Kousuke Ebihara13年以上前に更新

  • トラッカーBug(バグ) から Backport(バックポート) に変更
  • 対象バージョンOpenPNE 3.7.0 から OpenPNE 3.6beta5 に変更

#3 Kousuke Ebihara13年以上前に更新

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

#4 Kousuke Ebihara13年以上前に更新

  • トラッカーBackport(バックポート) から Bug(バグ) に変更
  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 対象バージョンOpenPNE 3.6beta5 から OpenPNE 3.7.0 に変更

誤ってトラッカーとステータスを変更してしまいました

#5 Kousuke Ebihara13年以上前に更新

http://redmine.openpne.jp/issues/1583 がこの件に関する opAuthMobileUIDPlugin 側の修正です

#6 Rimpei Ogawa13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

今回の変更により hasMobileUidCookie() が使用されなくなっているので、使う予定がなければ削除してください。

#7 Kousuke Ebihara13年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#8 Rimpei Ogawa13年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#9 Yuma Sakata12年以上前に更新

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

テストOKです。

#10 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

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