Project

General

Profile

Bug(バグ) #3168

OpenPNE.yml の ssl_base_url に api.php 向けの設定が存在しない

Added by Youichi Kimura over 7 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2012-08-22
Due date:
% Done:

0%

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

Description

Overview (現象)

OpenPNE をサブディレクトリに設置している環境でブラウザから SSL でアクセスすると、 openpne.apiBase が下記のようにサブディレクトリを考慮しない値になってしまう。

<script type="text/javascript">
//<![CDATA[

var openpne = {"apiKey":"98432...624ba","apiBase":"\/api.php\/"};

//]]>
</script>

例えば https://example.com/sns/ に OpenPNE を設置しているならば openpne.apiBase の値は \/sns\/api.php\/ となるのが正しい。

Causes (原因)

OpenPNE.yml の ssl_base_url は現在 pc_frontend, mobile_frontend, pc_backend 向けの設定項目があるが、api に対する項目は存在していない。アプリケーションを跨いだ URL を生成する際に opApplicationConfiguration::getAppRouting() の下記の箇所で $sslBaseUrls['api'] の値を取得しようとすると NULL が返ってしまうため、その先の処理で意図しない挙動を起こしてしまう。

    $options = $context->getRouting()->getOptions();
    if ($options['context']['is_secure'])
    {
      $sslBaseUrls = sfConfig::get('op_ssl_base_url');
      $url = $sslBaseUrls[$application];
      $isDefault = 'https://example.com' === $url;
    }
    else
    {
      $url = sfConfig::get('op_base_url');
      $isDefault = 'http://example.com' === $url;
    }

Way to fix (修正内容)

下記のように ssl_base_url に api アプリケーション向けの設定項目を OpenPNE.yml.sample に追加する。

# SSL用URL (Cookie の値を引き継ぐため、 base_url と同一ドメイン・パスである必要があります)
# Base URL for SSL (Set same domain and same path with "base_url" configuration to take over a value of Cookie)
ssl_base_url:
  pc_frontend: "https://example.com" 
  mobile_frontend: "https://example.com" 
  pc_backend: "https://example.com" 
  api: "https://example.com" 

Related issues

Related to OpenPNE 3 - Backport(バックポート) #3169: OpenPNE.yml の ssl_base_url に api.php 向けの設定が存在しない Fixed(完了) 2012-08-22

Associated revisions

Revision 32f97dda (diff)
Added by Youichi Kimura over 7 years ago

add ssl_base_url for `api' application in OpenPNE.yml.sample (fixes #3168)

History

#1 Updated by Yuma Sakata over 7 years ago

  • Target version set to OpenPNE 3.9.0-old

#2 Updated by Youichi Kimura over 7 years ago

  • Description updated (diff)

#3 Updated by Youichi Kimura over 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by Rimpei Ogawa about 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70
  • 3.8 で発生するか set to Unknown (未調査)

#6 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.2 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF