weiguoqiang/cert-manager
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #492 from munnerz/e2e-long-domain

Browse Source 
Add e2e test to ensure we can obtain certificates for dnsNames >100 chars
This commit is contained in:
jetstack-bot 2018-04-19 10:20:42 +01:00 committed by GitHub
commit 8485ada4e2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
test/e2e/certificate/certificate_acme.go
View File

@ -98,6 +98,15 @@ var _ = framework.CertManagerDescribe("ACME Certificate (HTTP01)", func() {
f.WaitCertificateIssuedValid(cert)
})
It("should obtain a signed certificate for a long domain using http01 validation", func() {
// the maximum length of a single segment of the domain being requested
const maxLengthOfDomainSegment = 63
By("Creating a Certificate")
cert, err := f.CertManagerClientSet.CertmanagerV1alpha1().Certificates(f.Namespace.Name).Create(util.NewCertManagerACMECertificate(certificateName, certificateSecretName, issuerName, v1alpha1.IssuerKind, acmeIngressClass, fmt.Sprintf("%s.%s", cmutil.RandStringRunes(maxLengthOfDomainSegment), util.ACMECertificateDomain)))
Expect(err).NotTo(HaveOccurred())
f.WaitCertificateIssuedValid(cert)
})
It("should obtain a signed certificate with a CN and single subdomain as dns name from the ACME server", func() {
By("Creating a Certificate")
cert, err := f.CertManagerClientSet.CertmanagerV1alpha1().Certificates(f.Namespace.Name).Create(util.NewCertManagerACMECertificate(certificateName, certificateSecretName, issuerName, v1alpha1.IssuerKind, acmeIngressClass, util.ACMECertificateDomain, fmt.Sprintf("%s.%s", cmutil.RandStringRunes(5), util.ACMECertificateDomain)))