Ticket #1108 (new バグ指摘)

Opened 9 years ago

Last modified 6 years ago

mtb_constants.php のみパーミッション 644

Reported by: Seasoft Owned by: somebody
Priority: Milestone: バックログ
Component: その他 Version: 2.11.0 β2
Keywords: Cc: Seasoft, shutta
修正済み: no

Description (last modified by kim) (diff)

/data/cache 配下の他ファイルは 666

関連)#988

Change History

comment:1 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.0 to EC-CUBE2.11.1(仮)

comment:2 Changed 9 years ago by kajiwara

  • 修正済み unset
  • Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.11.2

comment:3 Changed 9 years ago by kajiwara

  • Milestone changed from EC-CUBE2.11.2 to EC-CUBE2.11.3(仮)

comment:4 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.3 to EC-CUBE2.11.4(仮)

comment:5 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

comment:6 follow-up: ↓ 8 Changed 8 years ago by shutta

少しコードを追ってみました。
パーミションが異なるのは、 SC_Initial.phpのcreateCacheDir()にて、

umask(0)

が実行されている部分が原因でした。
これ以前に作成されたファイルは644、それ以降は666になります。(但し、Apacheの設定によって異なる)
なので、mtb_constants.php以外は、意図せず?666になってしまっているんじゃないかと思います。
本来は、SC_Utils::sfCopyDir()のようにumask値を保存しておいて、元に戻すようにするのが礼儀正しいと思いますので、 umaskを変更するような処理はついでに全て書き直してやろうかと思います。

で、本チケットの意図としては、パーミッションをどちらかに合わせたいというものだと推測しますが、
下記のどちらを想定していますでしょうか?
a) 644 に合わせる
b) 666 に合わせる

comment:7 Changed 8 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.12.2 to EC-CUBE 2.12.3

comment:8 in reply to: ↑ 6 ; follow-up: ↓ 9 Changed 8 years ago by kim

  • Cc Seasoft, shutta added
  • Milestone changed from EC-CUBE2.12.3 to EC-CUBE2.12.4

seasoft様、shutta様 shutta への返信

で、本チケットの意図としては、パーミッションをどちらかに合わせたいというものだと推測しますが、
下記のどちらを想定していますでしょうか?
a) 644 に合わせる
b) 666 に合わせる

こちら、644に合わせるのが望ましいと思われますが、なにか懸念される点等ありますでしょうか。

comment:9 in reply to: ↑ 8 Changed 8 years ago by Seasoft

環境依存なのか、現在別の環境(2.12.2)で見たら下記の状況でした。

  • mtb_constants.php のみ 777
  • 他は 666

こちら、644に合わせるのが望ましいと思われますが、なにか懸念される点等ありますでしょうか。

PHP を Apache のモジュールで使用している環境 (多分、VPS 利用者の大半が該当) で、FTP による編集ができなくなると思います。ただ、影響範囲が /data/cache/ 配下に限定されるならば、通常変更するようなファイルではないので、実務面でのインパクトは軽減するとも思います。

comment:10 Changed 7 years ago by Ringo

mtb_constants.php も、他の .serial 同様、変更時に 「削除」→「作成」(ロジック未確認)される様にすれば、644でも良いですが、 mtb_constants.phpのみ「上書き」(ロジック未確認)のままでは、 オーナーが httpdプロセス でない場合「パラメータ設定」の「登録」で、 dbのみ変更され、mtb_constants.phpが「変更」されず、

コミュニティでの『パラメータの変更が有効にならない』

が、解決されないです。(httpdのグループの書込権限付与で解決)

本チケットは「他と異なる」事が問題ではなく(今となっては?)↑の解決のためのチケットと理解しています。

comment:11 Changed 7 years ago by kim

  • Description modified (diff)

#988 とも併せて検討が必要ですね。いったん、2,13,0でおきます、

comment:12 Changed 7 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.13.0 to EC-CUBE 2.13.1

comment:13 Changed 7 years ago by m_uehara

  • Milestone changed from EC-CUBE 2.13.1 to EC-CUBE 2.13.2

comment:14 Changed 6 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.13.2 to EC-CUBE2.13.3

comment:15 Changed 6 years ago by kim

  • Milestone changed from EC-CUBE2.13.3 to バックログ

継続検討課題として、バックログへ移させていただきます

Note: See TracTickets for help on using tickets.