Cron has to be executed with the correct user

[DeepDiver1975]

the same approach as in #13282

We need to cleanup this - we are duplicating too much stuff here.

Choosing simple approach to allow backport

@karlitschek @MorrisJobke @schiesbn please review - THX

[scrutinizer-notifier]

A new inspection was created.

[DeepDiver1975]

fixes #15353, #15299 (comment)

[MorrisJobke]

Works 👍

[RobinMcCorkell]

The owner of config.php is not necessarily the web server user... Can you check something that is generated by ownCloud, like data/.ocdata?

[ghost]

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/11128/
🚀 Test PASSed.🚀

[MorrisJobke]

The owner of config.php is not necessarily the web server user... Can you check something that is generated by ownCloud, like data/.ocdata?

But it should.

[RobinMcCorkell]

@MorrisJobke Why? The requirement is that the web server user can write the config.php, not that it owns it.

[MorrisJobke]

@MorrisJobke Why? The requirement is that the web server user can write the config.php, not that it owns it.

true story ... sorry.

[DeepDiver1975]

This is the check we have in place for the console as well since oc8.
I'd prefer to get this in now as well and backport this - further enhancments (not to be backported) will go in a second step into master only.

@Xenopathic sounds reasonable?

[RobinMcCorkell]

@DeepDiver1975 OK, that sounds reasonable 😄

Code looks good 👍

[DeepDiver1975]

@karlitschek backport? THX

[karlitschek]

please backport. this is important

[DeepDiver1975]

@MorrisJobke @nickvergessen some spare time to prepare the PR for me? THX a lot!

[MorrisJobke]

stable8 #15369

[MorrisJobke]

stable7 #15370