-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Renamed file logging #523
Renamed file logging #523
Conversation
- Updated the config sample to point to log_type='file' - Renamed the Class for logfile logging to File in namespace 'OC\Log\'. Changed the occurrences of 'OC\Log\Owncloud' to 'OC\Log\File'. - Renamed the Class for log:file command to File in namespace 'OC\Core\Command\Log\File'. Changed registration of the command to use 'OC\Core\Command\Log\File'. - Changed default Syslog tag to Nextcloud - Retained backwards compatibility for configs with 'logtype' => 'owncloud' - Adjusted tests for the new file log. Closes nextcloud#490.
@Faldon, thanks for your PR! By analyzing the annotation information on this pull request, we identified @Xenopathic, @bartv2 and @LukasReschke to be potential reviewers |
I tested this and it works 👍 Also the code looks good |
@@ -22,7 +22,7 @@ | |||
namespace Tests\Core\Command\Log; | |||
|
|||
|
|||
use OC\Core\Command\Log\OwnCloud; | |||
use OC\Core\Command\Log\File; | |||
use Test\TestCase; | |||
|
|||
class OwnCloudTest extends TestCase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probabaly rename the test as well ;)
👍 |
Could backport to 10? |
cc @karlitschek |
backport makes sense 👍 |
Given the current situation I would like to not backport this to stable10 and only keep this in master. |
Changed file logging from 'owncloud' to 'file'
Closes #490