[stable9] Adding certificate revocation list and validate if the app certificate is revoked #25469

merged 2 commits into from Jul 13, 2016


None yet

3 participants


Backport of #25459 to stable9

Please review and test @DeepDiver1975 @owncloud/qa

For this, best is to patch the daily build with this PR and then try to enable an app that has a revoked cert. Also check that other valid apps from the store (like mail) can be enabled properly.

DeepDiver1975 added some commits Jul 12, 2016
@DeepDiver1975 @PVince81 DeepDiver1975 Adding certificate revocation list and validate if the app certificat…
…e is revoked
@DeepDiver1975 @PVince81 DeepDiver1975 Check integrity of a signed app in any case on installation
@PVince81 PVince81 added this to the 9.0.4 milestone Jul 13, 2016

@PVince81, thanks for your PR! By analyzing the annotation information on this pull request, we identified @LukasReschke, @georgehrke and @icewind1991 to be potential reviewers


Tested by patching daily stable9. Enabling or updating to an app with revoked cert is blocked.
Works as expected.


tested 👍

@DeepDiver1975 DeepDiver1975 merged commit fb2d273 into stable9 Jul 13, 2016

4 of 5 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
cla-bot-core Build #5313 succeeded in 5 min 56 sec
continuous-integration/php-5.4 Build #5750 succeeded in 50 min
server-master-linux-externals-smb-windows-ext-ci/database=sqlite,external=smb-windows,label=master Build #24604 succeeded in 2 min 49 sec
server-master-linux-php7-ci/database=sqlite,label=SLAVE Build #40839 succeeded in 48 min
@DeepDiver1975 DeepDiver1975 deleted the stable9-revoke-app-dev-cert branch Jul 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment