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

Opened 12 years ago

Closed 11 years ago

#2030 closed defect (fixed)

サイズ上限を超えた画像を添付して日記・トピックコメントをメール投稿すると、エラーメールが返されず本文のみが反映されている

Reported by: imamura623 Owned by: ebihara
Priority: critical Milestone: OpenPNE2.10.6
Component: pne-ktai Version: 2.8.x & 2.10.x & 2.11.x
Keywords: OpenPNE2.8.11 OpenPNE2.12beta2 Cc:

Description

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

サイズ上限を超えた画像を添付して、携帯メールから日記を投稿すると、
画像の掲載は処理されず、日記の本文だけが掲載されます。

sns.phpを見ていると、コミュニティなどへのメール投稿は、

 『写真が添付されていないか、ファイルサイズが大きすぎるため、登録できませんでした。』

といったエラーメッセージをユーザに返しており、
本来、日記に関してもこれと同様にすべきではないのでしょうか。

現状、get_images()が、

 ・画像が添付されていない(場合によりエラー)
 ・画像ではない、あるいは処理できない画像(常にエラー)
 ・サイズオーバー(常にエラー)

という別の判断に対して同じ結果(空の配列)を返していますが、
ここがどうも筋として違う気がしています。

いかがでしょうか?

とりあえず、get_images()を書き換えてやりくりしようとは考えていますが・・・
追記
バージョン2.10.0 です

Change History (29)

comment:1 Changed 12 years ago by kiwa

Priority: minormajor

comment:2 Changed 12 years ago by kiwa

Keywords: 再現待ち removed
Summary: サイズ上限を超えた画像を添付して日記をメール投稿すると、エラーメールが返されず本文のみが反映されているサイズ上限を超えた画像を添付して日記・トピックコメントをメール投稿すると、エラーメールが返されず本文のみが反映されている

以下のような結果になりました

■2.8.x

本文なし 画像容量オーバー
日記 エラーメールあり エラーメール無し
トピック エラーメールあり エラーメールなし
プロフィール画像 - -
トピック画像 - -
イベント画像 - -
コミュニティ画像 - -

■2.10.x

本文なし 画像容量オーバー
日記 エラーメールあり エラーメール無し
トピック エラーメールあり エラーメールなし
プロフィール画像 -
トピック画像 -
イベント画像 -
コミュニティ画像 -

■2.11.x

本文なし 画像容量オーバー
日記 エラーメールあり エラーメール無し
トピック エラーメールあり エラーメールなし
プロフィール画像 -
トピック画像 -
イベント画像 -
コミュニティ画像 -

comment:3 Changed 12 years ago by kiwa

Priority: majorminor

comment:4 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 added
Milestone: OpenPNE2.10.5

comment:5 Changed 12 years ago by kiwa

Priority: minormajor

comment:6 Changed 12 years ago by ebihara

Priority: majorcritical

comment:7 Changed 12 years ago by kiwa

日記・コミュニティ書き込みでIMAGE_MAX_FILESIZEで設定した容量以上の添付ファイルのついたメール投稿が行われた場合、以下のエラーメールを返して、日記本文も反映させないようにしましょう。

ファイルサイズが大きすぎるため、登録できませんでした。

comment:8 Changed 12 years ago by kiwa

Priority: criticalmajor

comment:9 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.5
Priority: majorminor

2.10.5では見送ります

comment:10 Changed 11 years ago by ebihara

Keywords: OpenPNE2.8.11 OpenPNE2.12beta2 added
Milestone: OpenPNE2.10.6

comment:11 Changed 11 years ago by ebihara

Priority: minorcritical

comment:12 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります

comment:13 Changed 11 years ago by ebihara

後回しにします。おそらく OpenPNE_KtaiMail::get_images() 周りの処理を書き換える必要があります。

comment:14 Changed 11 years ago by ebihara

Owner: changed from ebihara to nobody
Status: assignednew

comment:15 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

対応方法を検討します。

comment:16 Changed 11 years ago by ebihara

条件分岐部分で型を考慮した比較をする必要がありますが、添付された画像のどれかにエラーがあった場合、OpenPNE_KtaiMail::get_images() は空配列ではなく false を返す形でとりあえずの対処をおこなおうと思います。

comment:17 Changed 11 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

comment:18 Changed 11 years ago by ogawa

Keywords: 確認中 removed

OpenPNE_KtaiMail::_get_images()

// 画像が正しいかどうかチェック
switch (strtolower($mail->ctype_secondary)) {
    ...
}

の部分のチェックがエラーだった場合にfalseを返さないようになっているように見えます。

それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。

comment:19 Changed 11 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで画像の形式チェックに失敗した場合にエラーを返すようにしました。

それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。

については、別チケットで開発版のみの改善項目として対応したと思います。

comment:20 Changed 11 years ago by ebihara

それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。

については、 #2368 でおこなうようにしました。

comment:21 Changed 11 years ago by ogawa

テストに回します。なるべく多くの実機でメール投稿試したいところですね。

comment:22 Changed 11 years ago by ogawa

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

comment:23 Changed 11 years ago by kiwa

Keywords: テスト待ち removed

以下の現象の確認をお願いします。

■Version

  • 2.8.x

■場所

  • 携帯版プロフィール写真変更(ktai_page_h_config_image)

■現象

  1. プロフィール写真投稿メールをIMAGE_MAX_FILESIZE以上の画像を添付した状態で送信した場合、画像は反映されないがエラーメールが返ってこない
  2. プロフィール写真投稿メールを画像が1枚も添付されていない状態で送信してもエラーメールが返ってこない
  3. プロフィール写真投稿メールにJPEG・GIF・PNG以外のファイルを添付して送信した場合、ファイルは反映されないがエラーメールが返ってこない

※上記3つの現象は、2.10.xと2.12betaXではエラーメールが返ってきます

comment:24 Changed 11 years ago by ebihara

プロフィール写真変更で登録できない写真を投稿した際にエラーメールを返す機能は 2.10beta1 で #1367 にて実装されたものであり、2.8.x ではこの機能が存在せず、コードレベルでは正しい動作であるといえます。

ただ変更が微小であり、また 2.10.x でこの変更についての不具合等が発生した事例がないことから、本チケットの範囲から外れますが #1367 でおこなった変更を取り込む形で対処しようと思います。

comment:25 Changed 11 years ago by ebihara

Keywords: 確認中 added

r7165で対応しました。ご確認ください。

comment:26 Changed 11 years ago by ogawa

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

2.12系のみですが、add_diary_commentの画像エラー処理が抜けているように見えます。

comment:27 Changed 11 years ago by ebihara

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

以下のリビジョンで add_diary_comment にも画像エラー処理を追加しました。

comment:28 Changed 11 years ago by ogawa

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

comment:29 Changed 11 years ago by kiwa

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

確認しました。開発版のテストはbetaでやるとして、こちらは閉じます。

Note: See TracTickets for help on using tickets.