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

Opened 14 years ago

Closed 13 years ago

#1867 closed defect (fixed)

管理画面の外部ログインフォーム用HTMLが部分SSL環境を考慮していない

Reported by: ebihara Owned by: akamine
Priority: trivial Milestone: OpenPNE2.12.6
Component: pne-admin Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.12 OpenPNE2.13.4 Cc:

Description (last modified by akamine)

管理画面の外部ログインページサンプルHTMLとパスワード再発行ページへのリンクサンプルHTMLが部分SSL環境を考慮していないので修正する。

■現象

config.php内の

特定ページだけのSSLを使用するかどうか

define('OPENPNE_USE_PARTIAL_SSL', false);

をtrueに変更しても外部ログインページサンプルのurlが変わらない。

■原因

edit_c_admin_config.tpl内の

<form action="({$smarty.const.OPENPNE_URL})" method="post" utn>

の部分でOPENPNE_URLを読みだしてしまっている為。

■修正内容

不具合対応用にedit_c_admin_config.phpでurl生成した為、リンクサンプルHTMLが部分SSL環境を考慮するようになった。

(今までdefine('OPENPNE_USE_PARTIAL_SSL', false)

をtrueに変更してもurlがOPENPNE_URLを呼んでいたものを、

trueに変更し、$GLOBALS_OPENPNE_SSL_REQUIREDに記述があれば、

urlが変更されるように修正しました。)

■関連情報

Change History (30)

comment:1 Changed 14 years ago by ogawa

Keywords: OpenPNE2.11.5 added; OpenPNE2.11.4 removed

comment:2 Changed 14 years ago by ebihara

Priority: majorminor

comment:3 Changed 14 years ago by kiwa

Keywords: OpenPNE2.8.9 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.4

2.10.4では見送りになりました。

comment:4 Changed 14 years ago by kiwa

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

comment:5 Changed 14 years ago by ebihara

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

今回のリリース項目からははずします。

comment:6 Changed 13 years ago by ogawa

Version: 2.8.x & 2.10.x & 2.11.x2.10.x & 2.12.x & 2.13.x

comment:7 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.7 added
Milestone: OpenPNE2.12RC1

comment:8 Changed 13 years ago by ebihara

Priority: minortrivial

comment:9 Changed 13 years ago by ogawa

Keywords: OpenPNE2.12RC1 added; OpenPNE2.10.7 removed
Milestone: OpenPNE2.12RC1OpenPNE2.10.7

comment:10 Changed 13 years ago by ebihara

Keywords: OpenPNE2.12RC2 added; OpenPNE2.12RC1 removed

comment:11 Changed 13 years ago by ogawa

Keywords: OpenPNE2.10.7 added; OpenPNE2.12RC2 removed
Milestone: OpenPNE2.10.7OpenPNE2.12.1

安定版マイナーバージョンアップリリースに回します。

comment:12 Changed 13 years ago by ogawa

Keywords: OpenPNE2.13.2 added

comment:13 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.8 added; OpenPNE2.10.7 removed
Milestone: OpenPNE2.12.1OpenPNE2.12.2

comment:14 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.9 added; OpenPNE2.10.8 removed
Milestone: OpenPNE2.12.2OpenPNE2.12.3

comment:15 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.10 added; OpenPNE2.10.9 removed
Milestone: OpenPNE2.12.3OpenPNE2.12.4

comment:16 Changed 13 years ago by kiwa

Keywords: OpenPNE2.13.3 added; OpenPNE2.13.2 removed

comment:17 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.10 OpenPNE2.13.3 removed
Milestone: OpenPNE2.12.4

comment:18 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.12 OpenPNE2.13.4 added
Milestone: OpenPNE2.12.6

comment:19 Changed 13 years ago by akamine

Owner: changed from nobody to akamine
Status: newassigned

自分が担当します。

comment:20 Changed 13 years ago by akamine

Description: modified (diff)
Keywords: 確認中 added

以下のリビジョンを修正しました。

comment:21 Changed 13 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • この変更では、 config.php の $GLOBALS_OPENPNE_SSL_REQUIRED? の要素に以下を追加しない限り、外部ログインページサンプルのURLがSSL用のURLになりません。
    '' => array(),
    
  • 報告にはパスワード再発行ページへのリンクサンプルにも同様の現象が起こるとありますが、こちらについてはどうでしょうか

comment:22 Changed 13 years ago by akamine

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed
Summary: 管理画面の外部ログインページサンプルHTMLとパスワード再発行ページへのリンクサンプルHTMLが部分SSL環境を考慮していない管理画面の外部ログインページサンプルHTMLへのリンクサンプルHTMLが部分SSL環境を考慮していない

以下のリビジョンを修正しました。

comment:23 Changed 13 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • openpne_gen_url_head() の第三引数のデフォルト値は true なので、 $absolute の指定の必要はありません。
  • 以下の true の直後に不要な半角スペースが存在します。
     18         $absolute = true ;
    
  • 以下の行がハードタブではじまっています。インデントには半角スペース4つを用いてください。
     16     //外部ログインフォーム用HTML内のアドレス生成
    
  • 以下の行末に不要な半角スペースがあります
     17         //PC版ログインフォーム 
    

comment:24 Changed 13 years ago by akamine

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

以下のリビジョンを再度修正しました。

comment:25 Changed 13 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • #comment:23 の指摘は openpne_gen_url_head() に対してのものです。Smarty 関数の t_url については、デフォルトで absolute に false を指定したうえで openpne_gen_url() をコールしているため、 comment:24 の修正は適切ではありません
  • ついでに、コミットメッセージの「差し戻しの個所を修正」というような表記はやめてください。コミットメッセージでは修正の内容を説明してください

comment:26 Changed 13 years ago by akamine

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

以下のリビジョンを再度修正しました。

comment:27 Changed 13 years ago by shingo

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

comment:28 Changed 13 years ago by shingo

Summary: 管理画面の外部ログインページサンプルHTMLへのリンクサンプルHTMLが部分SSL環境を考慮していない管理画面の外部ログインページサンプルHTMLとパスワード再発行ページへのリンクサンプルHTMLが部分SSL環境を考慮していない

Summary変更

comment:29 Changed 13 years ago by kiwa

Summary: 管理画面の外部ログインページサンプルHTMLとパスワード再発行ページへのリンクサンプルHTMLが部分SSL環境を考慮していない管理画面の外部ログインフォーム用HTMLが部分SSL環境を考慮していない

comment:30 Changed 13 years ago by kiwa

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

確認しました。問題ありません。

Note: See TracTickets for help on using tickets.