tpay-com / tpay-php

MIT License
24 stars 27 forks source link

[CRITICAL] Commit "Extract logging logic outside Util class" psuje istniejące integracje i ignoruje customowy config logowania działań #64

Closed hoRacy closed 10 months ago

hoRacy commented 10 months ago

Witajcie. Po commicie Extract logging logic outside Util class (https://github.com/tpay-com/tpay-php/pull/60[)](https://github.com/tpay-com/tpay-php/commit/e0741879d50f3ea626c056cfb5e68981ced88766) istniejące integracje z customowym logowaniem błędów przestają działać.

Jest to breaking change i nie powinno mieć miejsca bez uprzedniego zmiany Major Versioning.

Ignorowane są ustawienia Util::$loggingEnabled Util::$customLogPatch

Ich wykorzystanie w kodzie tpay-com/tpay-php zostało usunięte, przez co paczka próbuje zapisać domyślnie logi w vendorach, gdzie www-data nie ma dostępów do zapisu. Zmienne te nadal istnieją w README.md.

Widzę, że w kodzie jest nowa zmienna: FileLogger::$logFilePath z tym że jest ona nieudokumentowana. Brakuje w ogóle alternatywy dla $loggingEnabled .

@s4ddly @kwojc

Pozdrawiam.