Skip to content

[stable12] Handle SSL certificate verifications for others than Let's Encrypt - #8184

Merged
MorrisJobke merged 1 commit into
nextcloud:stable12from
robert-scheck:pki-validation-stable12
Feb 8, 2018
Merged

[stable12] Handle SSL certificate verifications for others than Let's Encrypt#8184
MorrisJobke merged 1 commit into
nextcloud:stable12from
robert-scheck:pki-validation-stable12

Conversation

@robert-scheck

@robert-scheckrobert-scheck commented Feb 5, 2018

Copy link
Copy Markdown
Contributor

Backport of #8182

Do no longer (wrongly) rewrite URLs like

for automated SSL certificate verifications. All (common commercial) certificate authorities (CA) except Let's Encrypt (via ACME) seem to use "pki-validation" rather "acme-challenge" for their domain control validation (DCV).

Signed-off-by: Robert Scheck robert@fedoraproject.org

Do no longer (wrongly) rewrite URLs like
* http://example.net/.well-known/pki-validation/file.txt (Comodo)
* http://example.net/.well-known/pki-validation/fileauth.txt (DigiCert, Thawte, GeoTrust)
* http://example.net/.well-known/pki-validation/gsdv.txt (GlobalSign)
* http://example.net/.well-known/pki-validation/starfield.htm (Starfield, GoDaddy)
* http://example.net/.well-known/pki-validation/swisssign-check.txt (SwissSign)
for automated SSL certificate verifications. All (common commercial)
certificate authorities (CA) except Let's Encrypt (via ACME) seem to
use "pki-validation" rather "acme-challenge" for their domain control
validation (DCV).
Signed-off-by: Robert Scheck <robert@fedoraproject.org>
@robert-scheckrobert-scheck changed the title Handle SSL certificate verifications for others than Let's Encrypt[stable12] Handle SSL certificate verifications for others than Let's EncryptFeb 5, 2018
@codecov

codecovBot commented Feb 5, 2018

Copy link
Copy Markdown

Codecov Report

Merging #8184 into stable12 will increase coverage by <.01%.
The diff coverage is 0%.

@@ Coverage Diff @@## stable12 #8184 +/- ##
==============================================
+ Coverage 53.83% 53.84% +<.01% 
Complexity 22761 22761 ==============================================
Files 1385 1385 Lines 87150 87150 Branches 1331 1331 ==============================================
+ Hits 46920 46925 +5 + Misses 40230 40225 -5
Impacted FilesCoverage ΔComplexity Δ
lib/private/Setup.php12.39% <0%> (ø)50 <0> (ø)⬇️
core/js/js.js61.83% <0%> (+0.55%)0% <0%> (ø)⬇️

@MorrisJobke
MorrisJobke merged commit e328ced into nextcloud:stable12Feb 8, 2018
@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

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@robert-scheck@nickvergessen@MorrisJobke