Improve CertificateManager to not be user context dependent

[MorrisJobke]

• removes the ability for users to import their own certificates (for external storage)
• reliably returns the same certificate bundles system wide (and not depending on the user context and available sessions)

The user specific certificates were broken in some cases anyways, as they are only loaded if the specific user is logged in and thus causing unexpected behavior for background jobs and other non-user triggered code paths.

• think about existing user specific certificates (maybe detect them and show them in the setup warnings so that the admin either imports them into the system wide chain or drops them)

[ChristophWurst]

🐘

[rullzer]

think about existing user specific certificates (maybe detect them and show them in the setup warnings so that the admin either imports them into the system wide chain or drops them)

Yeah so the mroe I think about the harder it is.
A setup check is most likely to slow. Maybe a background job that collects this info. And then shows it in the setupchecks if needed. Then we have an occ command to migrate the certificates? Or we just say. Well to bad. REmove the user thing. Add a release note and move on.

[rullzer]

Rebases and pushed another commit to fix other parts of the system as well.

[MorrisJobke]

While digging through the code I found that the controller was removed in #21224. I will check when the routes and UI was removed to give some estimates on the usage.

[MorrisJobke]

While digging through the code I found that the controller was removed in #21224. I will check when the routes and UI was removed to give some estimates on the usage.

The settings part was removed in 13: #4890. So let's just check if there are user certificates and then show this to the admin.

[MorrisJobke]

Fixed the tests, added a one time background job, added texts for the setup checks and rebased and squashed everything. Ready for review IMO. I also added a screenshot to the OP.

[MorrisJobke]

Fixed the tests. Ready for review 🚀

[faily-bot]

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 34946: failure

mysql5.6-php7.3

Show full log

There were 2 warnings:
1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
--
There was 1 failure:
1) Test\Files\Cache\Wrapper\CacheJailTest::testWithoutNormalizer
Failed asserting that 0 is greater than 0.
/drone/src/tests/lib/Files/Cache/CacheTest.php:554