DevFW-CICD/ingress-nginx-helm
14
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #4249 from ElvinEfendi/trailing-dot

Browse source 
test to make sure dynamic cert works trailing dot in domains
This commit is contained in:
Kubernetes Prow Robot 2019-06-29 16:43:22 -07:00 committed by GitHub
commit a2009484f7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
test/e2e/lua/dynamic_certificates.go
View file

@ -130,6 +130,10 @@ var _ = framework.IngressNginxDescribe("Dynamic Certificate", func() {
ensureHTTPSRequest(f.GetURL(framework.HTTPS), host, host) ensureHTTPSRequest(f.GetURL(framework.HTTPS), host, host)
}) })
It("supports requests with domain with trailing dot", func() {
ensureHTTPSRequest(f.GetURL(framework.HTTPS), host+".", host)
})
It("picks up the updated certificate without reloading", func() { It("picks up the updated certificate without reloading", func() {
ing, err := f.KubeClientSet.ExtensionsV1beta1().Ingresses(f.Namespace).Get(host, metav1.GetOptions{}) ing, err := f.KubeClientSet.ExtensionsV1beta1().Ingresses(f.Namespace).Get(host, metav1.GetOptions{})
Expect(err).ToNot(HaveOccurred()) Expect(err).ToNot(HaveOccurred())