ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 8 years ago

Closed 8 years ago

#4166 closed defect (fixed)

Amazon仕様変更後のレビュー機能がPHP4環境で動作しない

Reported by: imamura623 Owned by: urabe
Priority: blocker Milestone: OpenPNE2.14.1.1
Component: 指定しない Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.12.14.1 確認中 Cc:

Description (last modified by urabe)

■現象

PHP4の環境(mhashが利用できない環境)で、Amazonのレビュー機能が利用できない

2009/08/21 追記

PHP4 でsha256 関数が使えないので、signature が空になり、Amazonから404エラーが返ってくる。2.14.1で行った変更ではPHP4の対応が完全に行えていない。

■原因

mhashが利用できない環境で、mhash関数が使われていたため、レビューが機能していなかった

2009/08/21 追記

  • PHP4 でsha256 が使えないので、signature が空になっていた

■修正内容

PEARライブラリのPHP_Compatを利用し、関数が定義されていない場合は、関数を読み込むよう修正

2009/08/21 追記

■関連情報

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=21190より転記

タイトルの環境で、レビューを書こうとして、キーワードで検索しても検索結果が出てきません。
いろいろと調べたところ、mhashもhash_hmacのどちらもないためにAmazon.co.jpにクエリが送信できていないようでした。

webapp/init.inc

に、
include_once 'PHP/Compat/Function/mhash.php';
と追記すると正常に動作しました。 

■関連チケット

Change History (31)

comment:1 Changed 8 years ago by imamura623

Description: modified (diff)

comment:2 Changed 8 years ago by urabe

Owner: changed from nobody to urabe
Status: newassigned

comment:3 Changed 8 years ago by kiwa

Keywords: OpenPNE2.12.14 added; 再現待ち removed
Milestone: OpenPNE2.14.1
Version: 2.12.x2.12.x & 2.14.x

早めに対応しましょう。

comment:4 Changed 8 years ago by kiwa

Summary: 2.12.13のレビュー機能がphp4.4.2で動作しないAmazon仕様変更後のレビュー機能がPHP4環境で動作しない

comment:5 Changed 8 years ago by kiwa

Priority: minorcritical

comment:6 Changed 8 years ago by urabe

Description: modified (diff)

以下のリビジョンでコミットしました。

comment:7 Changed 8 years ago by urabe

Keywords: 確認中 added

comment:8 Changed 8 years ago by shingo

他の箇所にあわせて、以下のように手動でコンポーネントをロードするやり方でもいいかと思います(修正するかどうかはお任せします)

require_once 'PHP/Compat/Function/mhash.php'

comment:9 Changed 8 years ago by shingo

Keywords: 差し戻し added; 確認中 removed

comment:10 Changed 8 years ago by shingo

今回問題となる mhash() は「Services/Amazon?.php」で使われている関数なので、その呼び出し箇所で

require_once 'PHP/Compat/Function/mhash.php'

とすればどうでしょう?
変更も最小限ですみます。

comment:11 Changed 8 years ago by urabe

Keywords: 確認中 added; 差し戻し removed

必要最小限の修正で済むように訂正しました

comment:12 Changed 8 years ago by shingo

Keywords: テスト待ち added; 確認中 removed

確認しました。

comment:13 Changed 8 years ago by imamura623

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

動作テストしました。問題無いようなので修正完了としこのチケットを閉じます。

comment:14 Changed 8 years ago by urabe

以下のリビジョンでAmazonパッチをコミットしました。

comment:15 Changed 8 years ago by kiwa

Resolution: fixed
Status: closedreopened

確認のため一度reopenします

comment:16 Changed 8 years ago by kiwa

Keywords: 確認中 added

comment:17 Changed 8 years ago by shingo

Keywords: 差し戻し added; 確認中 removed

確認しました。 テストにまわしますが、前回の修正版を含んだ「完全パッチ」があればよいと思います

comment:18 Changed 8 years ago by urabe

#3982 #4166 Amazon の仕様変更の修正、PHP4での動作修正を含めたパッチをコミットしました。 r12622

comment:19 Changed 8 years ago by urabe

Keywords: 確認中 added; 差し戻し removed

comment:20 Changed 8 years ago by shingo

完全パッチ:AmazonECS4クラスが修正されなくなっています

comment:21 Changed 8 years ago by urabe

r12624 でコミットしました。

comment:22 Changed 8 years ago by shingo

Keywords: テスト待ち added; 確認中 removed

確認しました

comment:23 Changed 8 years ago by imamura623

Keywords: テスト待ち removed
Resolution: fixed
Status: reopenedclosed

パッチの動作確認しました。問題無いようなのでこのチケットを閉じます。

comment:24 Changed 8 years ago by urabe

Resolution: fixed
Status: closedreopened

comment:25 Changed 8 years ago by urabe

Amazon レビュー機能に完全に対応ができていなかったので、チケットをリオープンしました。

  • PHP4 でsha256 関数が使えないので、signature が空になり、Amazonから404エラーが返ってくる現象を確認

comment:26 Changed 8 years ago by kiwa

Description: modified (diff)
Keywords: OpenPNE2.12.14.1 added; OpenPNE2.12.14 removed
Milestone: OpenPNE2.14.1OpenPNE2.14.1.1
Priority: criticalblocker

既にAmazon仕様変更の移行期間は終了しています。

該当の環境でレビュー機能が使用できない状態になっているので、緊急対応を行います。

comment:27 Changed 8 years ago by urabe

以下のリビジョンでコミットしました。

パッチファイルをコミットしました。

  • (2.10.x) すべての Amazon 対応を包括したパッチ
    • OpenPNE_2_10_x_Amazon_ticket_3982_4166_2_fix.patch
  • (2.10.x) 今回の修正分のみの Amazon 対応パッチ
    • OpenPNE_2_10_x_Amazon_ticket_4166_2_fix.patch
  • (2.8.x) すべての Amazon 対応を包括したパッチ
    • OpenPNE_2_8_x_Amazon_ticket_3982_4166_2_fix.patch
  • (2.8.x) 今回の修正分のみの Amazon 対応パッチ
    • OpenPNE_2_8_x_Amazon_ticket_4166_2_fix.patch

comment:28 Changed 8 years ago by urabe

patch ファイルは以下のリビジョンでコミットしています。

comment:29 Changed 8 years ago by urabe

Description: modified (diff)

comment:30 Changed 8 years ago by urabe

Keywords: 確認中 added

comment:31 Changed 8 years ago by imamura623

Resolution: fixed
Status: reopenedclosed

動作テストしました。問題ありませんでしたのでチケットを閉じます。

Note: See TracTickets for help on using tickets.