Skip to content

[stable12] Update CRL because user_sql cert was lost - #8152

Merged
MorrisJobke merged 1 commit into
stable12from
12-8150
Feb 2, 2018
Merged

[stable12] Update CRL because user_sql cert was lost#8152
MorrisJobke merged 1 commit into
stable12from
12-8150

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

* see nextcloud/app-certificate-requests#129
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Feb 2, 2018
@MorrisJobkeMorrisJobke added this to the Nextcloud 12.0.6 milestone Feb 2, 2018
@nickvergessen

Copy link
Copy Markdown
Member

The diff here is different, was it forgotten last time?

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

The diff here is different, was it forgotten last time?

Seems like that.

@MorrisJobkeMorrisJobke changed the title Update CRL because user_sql cert was lost[stable12] Update CRL because user_sql cert was lostFeb 2, 2018
@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 2, 2018
@codecov

codecovBot commented Feb 2, 2018

Copy link
Copy Markdown

Codecov Report

Merging #8152 into stable12 will decrease coverage by <.01%.
The diff coverage is n/a.

@@ Coverage Diff @@## stable12 #8152 +/- ##
==============================================
- Coverage 53.84% 53.83% -0.01% 
Complexity 22761 22761 ==============================================
Files 1385 1385 Lines 87150 87150 Branches 1331 1331 ==============================================
- Hits 46925 46921 -4 - Misses 40225 40229 +4
Impacted FilesCoverage ΔComplexity Δ
lib/private/Security/CertificateManager.php91.08% <0%> (-1%)39% <0%> (ø)
core/js/js.js61.27% <0%> (-0.56%)0% <0%> (ø)
lib/private/Server.php84.8% <0%> (+0.12%)121% <0%> (ø)⬇️
lib/private/Files/Cache/Propagator.php96.2% <0%> (+1.26%)16% <0%> (ø)⬇️

@MorrisJobke
MorrisJobke merged commit 02fba90 into stable12Feb 2, 2018
@MorrisJobke
MorrisJobke deleted the 12-8150 branch February 2, 2018 12:34
@MorrisJobkeMorrisJobke mentioned this pull request Mar 9, 2018
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@MorrisJobke@nickvergessen@ChristophWurst