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

Opened 12 years ago

Closed 11 years ago

#1563 closed defect (fixed)

DoCoMoのメールアドレスが二重引用符が付いた状態で登録されてしまう場合がある

Reported by: ebihara Owned by: ebihara
Priority: critical Milestone: OpenPNE2.10.4
Component: pne-ktai Version: 2.8.x & 2.10.x & 2.11.x
Keywords: OpenPNE2.8.9 OpenPNE2.11.5 Cc:

Description

DoCoMoのメールアドレスには、以下のような二重引用符が付いた形式のものがある。

"example"@docomo.ne.jp

OpenPNEはこの二重引用符を取り除いてから登録処理をしており、メールアドレスを用いる箇所もそれを前提としてコーディングされている。

ところが二重引用符が取り除かれない状態で登録されているユーザが存在し、メール投稿が行えないなどの問題が発生している(二重引用符が取り除かずに登録処理をしている箇所がある?)とのこと。

Change History (16)

comment:1 Changed 12 years ago by ebihara

動作を確認してみたところ、以下のような結果になりました。

  • 二重引用符が取り除かれた状態で登録される
    • get@MAIL_SERVER_DOMAINから登録した場合
  • 二重引用符が取り除かれないまま登録される
    • ユーザ画面から招待した場合
    • 管理画面のCSVインポートから登録した場合
    • 管理画面からの招待

comment:2 Changed 12 years ago by ebihara

Keywords: 再現待ち removed
Owner: changed from nobody to ebihara
Status: newassigned

とりあえず、携帯メールアドレスを使用しての認証時(パスワードログイン、メール投稿)のみ、二重引用符が付いている場合・付いていない場合の両方をチェックするようにして対処したいと思います。

comment:3 Changed 11 years ago by ebihara

Owner: changed from ebihara to nobody
Status: assignednew

comment:4 Changed 11 years ago by ebihara

Keywords: OpenPNE2.8.9 OpenPNE2.11.5 added
Milestone: OpenPNE2.11.5

comment:5 Changed 11 years ago by ebihara

Milestone: OpenPNE2.11.5OpenPNE2.10.4

comment:6 Changed 11 years ago by ebihara

Priority: majorcritical

comment:7 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:8 Changed 11 years ago by ebihara

ログイン周りの処理が複雑で、認証失敗時にログイン処理をリトライするのは困難なので、以下の方針で対応しようと思います。

  • ログインについてはかんたんログインや、最悪の場合登録し直すといった策が取れるので、ここでは対処しない
  • メールアドレスをローカル部分が二重引用符つきのものとそうでないものと両方チェックするのは、メール投稿のみ
  • メールアドレス登録時には、必ず二重引用符を取り除くようにする

comment:9 Changed 11 years ago by ebihara

Keywords: 確認中 added

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

comment:10 Changed 11 years ago by ogawa

Keywords: 確認中 removed

OpenPNE_KtaiMail::_get_mail_address() で二重引用符を取り除く処理があるので、webapp/lib/mail/sns.php の「二重引用符がある場合」は考慮する必要はないのではないでしょうか。

comment:11 Changed 11 years ago by ebihara

Keywords: 確認中 added

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

comment:12 Changed 11 years ago by ogawa

Version: 2.8.x & 2.10.x & 2.11.x

comment:13 Changed 11 years ago by ogawa

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

comment:14 Changed 11 years ago by kiwa

Keywords: テスト待ち removed

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

■発生バージョン

  • 2.8.x
  • 2.10.x
  • trunk

■現象

  • 場所:ktai_page_o_login2
  • 内容:アカウントに二重引用符がついた状態のメールを登録しようとすると、o_login2の携帯メールアドレス表示部分に、二重引用符がついた状態のままの携帯メールアドレスが表示される

comment:15 Changed 11 years ago by ogawa

Keywords: テスト待ち added

c_ktai_address_preにINSERTする直前にも二重引用符を取り除く処理を追加してみました。確認お願いします。

comment:16 Changed 11 years ago by kiwa

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

確認しました。問題なさそうです。

Note: See TracTickets for help on using tickets.