Default for propfind depth infinity should be false as in the past

[mrow4a]

Change in https://github.com/owncloud/core/pull/38583/files#diff-efe547293d3c10c67f8014fd439977f1c8f1073dcb65a1a83dc33e7d266dec70R48 release for oc10.9 set dav.propfind.depth_infinity' => true as default and caused some performance issues on some setups. This PR brings back the default as in the past, as discussed in #38583 (comment).

• review unit tests for potential adjustments

fixes https://github.com/owncloud/enterprise/issues/5154

[ownclouders]

💥 Acceptance tests pipeline apiComments-mariadb10.2-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/core/35465/51

[mmattel]

Hooking myself in to get notified when merged so we can start a config-to-docs run.
@EParzefall FYI

[phil-davis]

Note: I am working on the acceptance tests in PR #40021 so that I don't make a mess of this PR.

I will first get them passing, then sort out adding a few scenarios to test:
a) what happens when a client does send a depth infinity request, but depth infinity is disabled
b) enabling depth infinity and a few test cases to be confident that it generally works

[jvillafanez]

The solution itself looks good, but I think we should clarify the doc in the config.sample

[mrow4a]

@phil-davis I think we should first have this change and then add acceptance on this basis.

[phil-davis]

@phil-davis I think we should first have this change and then add acceptance on this basis.

agree - we just need to do the minimal change to the acceptance tests so that the current tests pass. I will keep working on that in #40021

[phil-davis]

I rebased and pushed the test changes from #40021

IMO CI should pass and we can merge.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

33.3% Coverage
0.0% Duplication