Mage :: log () yeni Magento güncellemesi üzerinde çalışmıyor (1.9.4.1)

Bu yeni güncellemeden sonra (1.9.4.1) Mage :: log () çalışmıyor. Görünüşe göre, Zend_Validate_File_ExtensionMage.php adresindeki 819 satırında, dosyanın daha is_readable()önce var olup olmadığını kontrol ettiği yerde yapması gereken bir şey var. Tüm log()yöntemi önceki sürümüne geri çevirdim ve yeniden çalışıyor.

Bu sorunu bildirmek için Magento ekibiyle bağlantı kurabileceğim ana kanal nedir?

Resmi yama geliyor :) Hala resmi yama bekliyorum ... :(

piotrekkaminski 13 saat önce yorum yaptı

Bu, daha önceki versiyonlara taşınacak olan güncel resmi yamadır (bu en son çalışmalıdır) https://gist.github.com/piotrekkaminski/0596cae2d25bf467edbd3d3f03ab9f8f

^{Kaynak: https://github.com/OpenMage/magento-lts/pull/648#issuecomment-480941871}

SUPEE-11086 ile yamalar konusunda Magento ile araştırma ve etkileşime dayanarak bulduğum her şeyi özetleyeceğim. Ne yapılabilir:

GÜNCELLEME 2: Bir sonraki PATCH SUPEE-11155 - https://magento.com/security/patches/supee-11155 adresinde sorun çözüldü . Her zaman olduğu gibi yamayı uygulamadan önce olası sorunları kontrol edin - Güvenlik Düzeltme Eki SUPEE-11155 - Olası sorunlar? Teşekkürler harika yorum için Aad Mathijssen için gidiyor .

Güncelleme: EE versiyonuna talep üzerine resmi bir yama mevcuttur. Temel olarak, Piotr Kaminski'nin özü Magento'nun yama dosyasıyla sarılmış hali.

1. app/Mage.phpDüzeltme eki dosyasındaki değişiklikleri silin . Şimdiye kadar yaptığım şey bu.
   Artıları - günlüğe kaydetme eskisi gibi çalışır.
   Eksileri - Bir düzeltme eki dosyasını düzenlerken, günlüğe kaydetme olası bir istismardan korunmuyor (ancak bu çok düşük bir risk olmalıdır). Magento resmi bir düzeltme yayınladığında, geri almak ve orijinal düzenlenmemiş Düzeltme Ekini uygulamak zorunda kalacaksınız.
2. Piotr Kaminski'nin özüne dayanarak en üste bir yama daha ekle - https://gist.github.com/piotrekkaminski/0596cae2d25bf467edbd3d3f03ab9f8f . Piotr Kaminski, güvenlikten sorumlu Magento'nun bir parçası, bu yüzden doğrudan atın ağzından geliyor. Gist, Magento'da paylaşıldı ve muhtemelen SUPEE-11086 v1.1 olarak bitecek.
   Artıları - Bu Magento yolu
   Eksileri - Bunun resmi hale gelmesini beklemeniz veya sorumluluk almanız ve onu yama olarak paketlemeniz gerekecek, bu da resmi bir yama tamamlandığında geri dönmeniz gerekecek.
   Bu değişikliklerle orijinali düzenlemek için iki yama eklemek yerine hafif bir değişiklik olabilir.
3. Düzen Zend_Validate_File_Extension::isValidve kaldır dosya varlığı doğrulama. Magento LTS github'da uzun bir tartışma var - https://github.com/OpenMage/magento-lts/pull/648 . isValidYöntem kendinden bekleneni değil şeyler yapar, bu nedenle bazı üyeler düzeltmek için öneriyoruz. Bence bu iyi bir çözüm değil, evet, kod kötü, ama sonsuza dek oradaydı ve özel modüllerde / kodlarda kullanılabilir. Aksine, olabilecek en kötüsü dosyaların varlığı kontrol edilmez.
   Artıları - oldukça basit bir düzeltme
   Cons - bir kütüphane dosyasını değiştirir ve işlevselliğini değiştirir.
   Bunu özel bir düzeltme eki olarak veya tüm sınıfı localkod havuzuna yeniden yazarak uygulayabilirsiniz .

Düzeltme ekini düzenlemeyi seçtim ve bir v1.1 geldiğinde, düzeltilen düzeltme ekini geri döndüreceğim ve orijinal sürümü uygulayacağım ve bu düzeltmeden sonra. Bu bizim yapım sürecimize ve iç politikamıza çok yakışıyor, sizin için farklı olabilir. Seçtiğiniz ne olursa olsun, bu düzeltme ekini daha sonra uygulamak daha iyidir.

Topluluk Girdilerinden bir şey. Yeni bir Validator var Zend_Validate_File_Extension aşağıdaki gibi

https://github.com/brentwpeterson/magento-patches/blob/master/CE1.9/PATCH_SUPEE-11086_CE_1.9.4.0_v1-2019-03-26-03-05-04.sh#L183

"Çözüm düzeltme ekini düzenliyor ve yalnızca app / Mage.php dosyasındaki değişiklikleri kaldırıyor. Bu uygulamayı kesinlikle reddederim, ancak durum kritiktir".

Benim geçici çözüm kopyalamak oldu lib/Zend/Validate/File/Extension.phpiçin app/code/local/Zend/Validate/File/Extension.phpve kaldırmak gelen bu kod parçasını isValid()yöntemiyle:

    // Is file readable ?
    #require_once 'Zend/Loader.php';
    if (!Zend_Loader::isReadable($value)) {
        return $this->_throw($file, self::NOT_FOUND);
    }

Olacak ...

public function isValid($value, $file = null)
{
    if ($file !== null) {
        $info['extension'] = substr($file['name'], strrpos($file['name'], '.') + 1);
    } else {
        $info = pathinfo($value);
...

Magento 1.9.4.2 piyasaya sürüldüğünde tekrar kontrol ediyorum.

Aslında, dosyanın okunamıyor olması veya mevcut olmaması, dosya adının geçerli olmadığı anlamına gelmez, değil mi?

Çekirdek kodunu değiştirmemenizi ve bunun gibi bir güncelleme kullanmamanı öneririm ( https://gist.github.com/mehdichaouch/99c67298b5a65f81219c9b69942b6fe7 )

$installer->run("
    INSERT INTO `{$installer->getTable('core_config_data')}` (scope, scope_id, path, value)
    VALUES ('default', 0, 'dev/log/allowedFileExtensions', 'log,txt,html,csv')
    ON DUPLICATE KEY UPDATE value = 'log,txt,html,csv';
");

Alt klasörlerin içine günlük dosyaları yazma olanağını engelleyen başka bir konu daha var (Magento ekibinden kasıtlı olabilir). Örneğin:

Mage::log('Some log information', Zend_Log::DEBUG, 'somefolder/anotherfolder/somelogfile.log', true);

Daha önceki sürümlerde, bu çağrı konumda bir dosya oluşturacaktı:

/your-magento-app-root-folder/var/log/somefolder/anotherfolder/somelogfile.log

Ancak basename(), Mage::log()yöntemde bir işlev çağrısı olduğundan, dosya şu adrese yazılır:

/your-magento-app-root-folder/var/log/somelogfile.log.

İşte suçlanan kod app/Mage.php:

$file = empty($file) ?
    (string) self::getConfig()->getNode('dev/log/file', Mage_Core_Model_Store::DEFAULT_CODE) : basename($file);

Özellikle 1.9.4.1 ile ilgili olmasa bile, sorun son zamanlarda ortaya çıkmaya başladı (en son 1.9.3.x sürümlerinde) ve bazen aynı adı taşıyan birçok günlük dosyasıyla uğraşmanız gerektiğinde çok can sıkıcıdır ( ama başlangıçta farklı alt klasörlerde).

Bu kod parçası Magento ekibinden muhtemelen kasıtlı olduğu için, daha sonraki bir sürümde düzeltmeyi planlamanın olmadığını düşünüyorum, bu da ilk davranışını geri yüklemek için kesmek anlamına gelir ...