プロジェクト

全般

プロフィール

Backport(バックポート) #2120

設定ファイルを記述し忘れた場合に、招待メールに記載されたURLにアクセスできない

Shinichi Urabeほぼ13年前に追加. 12年以上前に更新.

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

100%


説明

http://sns.openpne.jp/communityTopic/6423 より転記

招待メールに記載されたURLにアクセスできません。

<動作環境>
OpenPNE3.6beta5
CentOS5(OpenVZの仮想環境)
PHP 5.2.11
mysql Ver 14.14

招待状に記載されていたURL
http://ドメイン名/member/register/token/3228758dcc32ffb79b61ee61deb73c1b1

実際に利用できた(手で書き換えた)URL
http://ドメイン名/sns/web/member/registerInput/token/3228758dcc32ffb79b61ee61deb73c1b1

登録完了メールには以下の正しいURLが記載されていました。
(このURLでOpenPNEにアクセス可能)
http://ドメイン名/sns/web/

本チケットで扱う問題

note-14 のコメントを以て、

  • 「設定ファイルを正しく設定したにもかかわらず、招待メールに記載されたURLにアクセスできない」という報告された問題は「再現せず」と判断する
  • 「設定ファイルを正しく設定していない場合に、招待メールに記載されたURLにアクセスできない」という問題に対応する

このようにこのチケットで扱う問題を切り替えます。具体的に何を問題として、何を修正するかについては note-9 以降を参照してください。

現象

【*NG】と示したものは、適切なURLが出力されていませんが、問題とみなす必要はないものです。

上記について説明を示します:

  • case1 は「設定ファイルを正しく記述した場合」で、適切なURLが出力されています
  • case2 は「設定ファイルを不正に記述した場合」で、URLは不適切ですが、これは設定ファイルを不正に記述していることが原因であるため問題とみなす必要はありません
  • case3, case4 は「設定ファイルを記述していない場合」で、招待メールのSNS参加リンクのドメインを見ると『ドメインを補完しようとしているが、サブディレクトリが抜け落ちている』という不自然な状態になっています

補足

サブディレクトリが抜け落ちると示していますが、厳密には「末尾のディレクトリ名が抜け落ちる」というものです。

note-9 で示されていますが、 dirname() 関数がパスの最後のディレクトリを除去してしまうという仕様に基づく問題です(dirname() の仕様が問題というよりは、当該部分で dirname() を用いていることが原因です)。

dirname('/usr/bin/ls') => '/usr/bin'
dirname('/usr/bin/') => '/usr'
dirname('/usr/bin') => '/usr'
dirname('/etc/') => '/'

対応すべき問題

case3, case4 の場合に、招待メールのURL(base_url相当部分)の補完が不十分であり、不自然なURLを生成してしまっているため、これを修正する。

  • 修正前の動作
    • 前提:サブディレクトリ(パス)を持つようなURLでSNSにアクセスする環境で、case3, case4 の場合、
    • 招待メールにおいて、SNS参加リンクのサブディレクトリ部分の末のディレクトリ名が抜け落ちる
  • 修正後の動作
    • 前提:サブディレクトリ(パス)を持つようなURLでSNSにアクセスする環境で、case3, case4 の場合、
    • 招待メールにおいて、SNS参加リンクが適切なURL(新規登録画面に遷移できるURL)となっている

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1675: 設定ファイルを記述し忘れた場合に、招待メールに記載されたURLにアクセスできない Fixed(完了) 2010-10-14
関連している OpenPNE 3 - Backport(バックポート) #2324: There are wrong URL generating for op_base_url with path name (パス名付きの op_base_url に対して誤った URL 生成をしている箇所がある) Fixed(完了) 2010-06-08 2011-10-06
関連している OpenPNE 3 - Backport(バックポート) #2325: OpenPNE.ymlのbase_url にパスが含まれない場合に、デバッグモードでwarningメッセージが表示される場合がある Fixed(完了) 2010-09-10 2011-10-06

履歴

#1 Minoru Takaiほぼ13年前に更新

  • 題名招待メールに記載されたURLにアクセスできない から 3.4.x 設定ファイルを記述し忘れた場合に、招待メールに記載されたURLにアクセスできない に変更
  • 優先度Urgent(急いで) から High(高め) に変更
  • 対象バージョンOpenPNE 3.4.13 から OpenPNE 3.4.14 に変更

Target version 3.4.13 で作成されていましたが、3.4.13 で取り込む理由がないため、元チケットが完了次第取り込むものとします。暫定的に Target version を 3.4.14 と変更します。

#2 Shinichi Urabeほぼ13年前に更新

  • 対象バージョンOpenPNE 3.4.14 から OpenPNE 3.4.15 に変更

親チケットで完了していないため、次期バージョンに延期します

#3 Minoru Takaiほぼ13年前に更新

  • 題名3.4.x 設定ファイルを記述し忘れた場合に、招待メールに記載されたURLにアクセスできない から 設定ファイルを記述し忘れた場合に、招待メールに記載されたURLにアクセスできない に変更

#4 Mutsumi Imamura12年以上前に更新

  • 担当者Mutsumi Imamura にセット

#5 Mutsumi Imamura12年以上前に更新

  • 担当者 を削除 (Mutsumi Imamura)

#6 Shinichi Urabe12年以上前に更新

  • 担当者Shinichi Urabe にセット

#7 Shinichi Urabe12年以上前に更新

  • 担当者 を削除 (Shinichi Urabe)
  • 対象バージョンOpenPNE 3.4.15 から OpenPNE 3.4.16 に変更

#2324#2325 の対応を併せて必要なため、今回の対応は見送ります

#8 Shinichi Urabe12年以上前に更新

  • 期日2011-10-06 にセット
  • 担当者Yuya Watanabe にセット

#9 Yuya Watanabe12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#10 Yuya Watanabe12年以上前に更新

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

本チケットは #2324 のBP元である #1155 の修正から発生した問題であると思われるため,下記コミットで同時に修正を行いました.

624162b8

参考:http://redmine.openpne.jp/issues/2324#note-2

#11 Minoru Takai12年以上前に更新

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

note-10 について

#2120 #2325 は下記実装案では問題として発生しないと考えられますが,レビューおよびテストの手がかりとするため,本チケット対応後に同様のステータスに変更します.

ということなので、本チケットにコミットが紐付いていませんが、本チケットは #2324 の重複チケットと見做さず、このチケットで扱う予定だったコミットを #2324 で併せて行った(コミットに紐付くチケットを変更している状態)と見做します(※)。

※通常、修正前には、修正後に直っているはずの問題が再現できるはずですが、関連修正を併せて行なっているためこれが確認できない可能性があります。このように結果的にコミットが紐付かないチケットが生じた場合は重複チケットとして(関連チケットで duplicates - duplicated by の紐付けをした上で) Duplidated (Invalid) クローズすることがしばしばあります。

本件のレビューは #2324624162b8 に対して行ったこととします。レビューOKです。

#12 Mutsumi Imamura12年以上前に更新

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

動作確認OKです。

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