[stable9] Log more information by default #23464

Merged
merged 2 commits into from Mar 23, 2016

Projects

None yet

4 participants

@LukasReschke
Member

Backport of #23443

@LukasReschke LukasReschke Log more information by default
This modifies the logger to add the following logging information by default:

- Request Method
- Request URL
- Current user
177ad39
@mention-bot

By analyzing the blame information on this pull request, we identified @MorrisJobke, @DeepDiver1975 and @Phiber2000 to be potential reviewers

@LukasReschke LukasReschke self-assigned this Mar 22, 2016
@MorrisJobke
Member

Tested and works 👍

@MorrisJobke
Member

Rebased to retrigger CI

@MorrisJobke
Member

@LukasReschke Why "2 - Developing"? I checked it with the command line tool, cron and the web UI all work.

@DeepDiver1975
Member

This change is suspected to break ci on mysql

@LukasReschke
Member

@LukasReschke Why "2 - Developing"? I checked it with the command line tool, cron and the web UI all work.

Because of #23488 😉

@MorrisJobke
Member

Because of #23488

Could you add that fix to this PR?

@LukasReschke LukasReschke Only use the user session if ownCloud is already installed
When installing ownCloud with autotest and MySQL some log entries may be created which will invoke the logging class. IUserSession has a dependency on the database which will make the installation fail => 💣
43516eb
@LukasReschke
Member

Could you add that fix to this PR?

Done.

@DeepDiver1975
Member

👍

@DeepDiver1975 DeepDiver1975 merged commit 0009bf2 into stable9 Mar 23, 2016

22 checks passed

cla-bot-core Build #2555 succeeded in 38 sec
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
core-ci-linux-jsunit/database=sqlite,label=SLAVE Build #59869 succeeded in 29 sec
Details
core-ci-linux-swift-primary-storage/database=mysql,label=SLAVE Build #53823 succeeded in 9 min 24 sec
Details
core-ci-linux/database=mysql,label=SLAVE Build #28405 succeeded in 10 min
Details
core-ci-linux/database=oci,label=SLAVE Build #28405 succeeded in 27 min
Details
core-ci-linux/database=pgsql,label=SLAVE Build #28405 succeeded in 10 min
Details
core-ci-linux/database=sqlite,label=SLAVE Build #28405 succeeded in 5 min 28 sec
Details
ocs-api-integration-tests-ci Build #8772 succeeded in 7 min 26 sec
Details
server-master-linux-externals-ci/database=sqlite,external=smb-silvershell,label=SLAVE Build #8557 succeeded in 1 min 15 sec
Details
server-master-linux-externals-ci/database=sqlite,external=swift-ceph,label=SLAVE Build #8557 succeeded in 2 min 30 sec
Details
server-master-linux-externals-ci/database=sqlite,external=webdav-ownCloud,label=SLAVE Build #8557 succeeded in 2 min 41 sec
Details
server-master-linux-externals-smb-windows-ext-ci/database=sqlite,external=smb-windows,label=master Build #9242 succeeded in 2 min 22 sec
Details
server-master-linux-php5.4-ci/database=sqlite,label=SLAVE Build #1213 succeeded in 3 min 23 sec
Details
server-master-linux-php7-ci/database=sqlite,label=SLAVE Build #36662 succeeded in 2 min 22 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=litmus,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 9 min 34 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@0,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 44 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@1,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 42 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_shareLink,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 23 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePermissions,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 40 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationGroups,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 15 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationInsideGroups,mirallBranch=v2.0.2,slave=SMASH Build #12911 succeeded in 12 min
Details
@DeepDiver1975 DeepDiver1975 deleted the stable9-backport-23443 branch Mar 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment