Project

General

Profile

Bug(バグ) #2225

SSL 通信時の携帯電話個体識別番号の取得が明確に禁止されていない

Added by Kousuke Ebihara about 8 years ago. Updated almost 4 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-06-16
Due date:
% Done:

100%

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

Description

Overview (概要)

SSL 通信時の携帯電話個体識別番号の取得が明確に禁止されていない。

SSL 通信で得られた携帯電話個体識別番号のなかには、信頼できない値になりうるものがある。 OpenPNE では SSL 通信時に すべての種類の 携帯電話個体識別番号の取得を禁止する方向で統一して実装する。

OpenPNE 3.6 では部分 SSL 機能を正式に提供するようになるので、この修正は OpenPNE 3.6 のリリース前におこなう必要がある。

Way to fix (修正内容)

  • SSL 通信では OpenPNE は携帯電話個体識別番号を得られなかったものとして認識するようにした
    • あくまで携帯電話個体識別番号を取得しないようにするだけで、 Cookie 内に格納された ID は通常通り利用できる(=かんたんログインができなくなるわけではない)
  • 部分 SSL 機能を利用している際に、携帯電話個体識別番号を用いるアクションを実行した際、その通信が HTTPS でおこなわれている場合は、 HTTP へリダイレクトさせるようにした
    • 対象となるアクションは 'member/register', 'member/registerInput', 'member/registerEnd', 'member/configUID' である

Related issues

Related to OpenPNE 3 - Backport(バックポート) #2269: SSL 通信時の携帯電話個体識別番号の取得が明確に禁止されていない Fixed(完了) 2011-07-14

Associated revisions

Revision 370717bd (diff)
Added by Kousuke Ebihara about 8 years ago

added support to force redirect to HTTP if the action uses mobileUID and that is in the ssl_selectable_actions list (refs #2225)

Revision a818aa1d (diff)
Added by Kousuke Ebihara about 8 years ago

opWebRequest::getMobileUID() now doesn't return uid when the current connection is SSL -- for a security reason (refs #2225)

Revision dbe81886 (diff)
Added by Kousuke Ebihara about 8 years ago

added test for the opWebRequest::getMobileUID() in SSL connection (fixes #2225)

Revision 764706b3 (diff)
Added by Kousuke Ebihara about 8 years ago

added test for opExecutionFilter::needToRetriveMobileUID() and opExecutionFilter::handleSsl() (fixes #2225)

Revision 05098b98 (diff)
Added by Kousuke Ebihara about 8 years ago

s/retrive/retrieve/ig to changes in this ticket (refs #2225)

Revision ba357d13 (diff)
Added by Kousuke Ebihara about 8 years ago

additional s/retrive/retrieve/ig to opExecutionFilterTest because there are oversights in my 05098b98 commit (refs #2225)

Revision 57ec41bd (diff)
Added by Kousuke Ebihara about 8 years ago

added "member/registerMobileToRegisterEnd" to list of retriving-mobile-uid-actions (fixes #2225)

History

#1 Updated by Kousuke Ebihara about 8 years ago

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

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

#2 Updated by Kousuke Ebihara about 8 years ago

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

#3 Updated by Rimpei Ogawa about 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

retrieve を retrive と typo しています。

#4 Updated by Shingo Yamada about 8 years ago

  • 360対象 set to beta13

#5 Updated by Kousuke Ebihara about 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

05098b98b1e0e810e9407bc0c535691fed683d8fhttp://redmine.openpne.jp/issues/2225#note-3 の指摘事項に対応しました。ただし、このチケットによる変更箇所以外の typo (サードパーティ製ライブラリも含む)については修正をおこなっていません。

#6 Updated by Rimpei Ogawa about 8 years ago

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

修正確認しました。

#7 Updated by Rimpei Ogawa about 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

一旦テスト待ちにしましたが見直したところ、 test/unit/filter/opExecutionFilterTest.php に3箇所 "retrive" が残っていましたので差し戻します。

test/unit/filter/opExecutionFilterTest.php
24:  public function callNeedToRetrieveMobileUID($module, $action, $retriveUIDMode = 1, $parameters = array())
26:    opConfig::set('retrive_uid', $retriveUIDMode);

#8 Updated by Rimpei Ogawa about 8 years ago

http://redmine.openpne.jp/issues/2087#note-13 に関連して追加の指摘事項です。

member/registerMobileToRegisterEnd アクション内にも mobile_uid を取得する処理があるため、このアクションも「携帯電話個体識別番号を扱うアクション」のリストに追加する必要があります。

#9 Updated by Kousuke Ebihara about 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#10 Updated by Maki Takahashi about 8 years ago

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

note-7 および note-8 で指摘されている内容が
修正されていることを確認いたしました。

#11 Updated by Kousuke Ebihara about 8 years ago

  • Description updated (diff)

修正内容として示していた以下の記述は事実と反していたため削除しました(本機能は ssl_required_applications よりも優先されることはなく、その場合、 ssl_selectable_actions から指定を外した場合は、従来通り、 HTTP 通信が強制されることになるため)。

** これらのアクションへのアクセスであっても、 OpenPNE.yml の ssl_selectable_actions で指定されていない場合は、リダイレクトはおこなわれない

#12 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100
  • 3.6 で発生するか changed from Yes to Unknown (未調査)
  • 3.4 で発生するか set to Unknown (未調査)

テストOKです。

#13 Updated by kaoru n almost 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF