vendor: Update quic and lego/acme dependencies

Author: mholt

caddytls/client.go

@@ -240,26 +240,30 @@ func (c *ACMEClient) Obtain(name string) error {
 	for attempts := 0; attempts < 2; attempts++ {
 		namesObtaining.Add([]string{name})
 		acmeMu.Lock()
-		certificate, failures := c.acmeClient.ObtainCertificate([]string{name}, true, nil, c.config.MustStaple)
+		certificate, err := c.acmeClient.ObtainCertificate([]string{name}, true, nil, c.config.MustStaple)
 		acmeMu.Unlock()
 		namesObtaining.Remove([]string{name})
-		if len(failures) > 0 {
+		if err != nil {
 			// Error - try to fix it or report it to the user and abort
 
-			var errMsg string // combine all the failures into a single error message
-			for errDomain, obtainErr := range failures {
-				if obtainErr == nil {
-					continue
+			if failures, ok := err.(acme.ObtainError); ok && len(failures) > 0 {
+				// in this case, we can enumerate the error per-domain
+				var errMsg string // combine all the failures into a single error message
+				for errDomain, obtainErr := range failures {
+					if obtainErr == nil {
+						continue
+					}
+					errMsg += fmt.Sprintf("[%s] failed to get certificate: %v\n", errDomain, obtainErr)
 				}
-				errMsg += fmt.Sprintf("[%s] failed to get certificate: %v\n", errDomain, obtainErr)
+				return errors.New(errMsg)
 			}
 
-			return errors.New(errMsg)
+			return fmt.Errorf("[%s] failed to obtain certificate: %v", name, err)
 		}
 
 		// double-check that we actually got a certificate; check a couple fields
 		// TODO: This is a temporary workaround for what I think is a bug in the acmev2 package (March 2018)
-		// but it might not hurt to keep this extra check in place
+		// but it might not hurt to keep this extra check in place (April 18, 2018: might be fixed now.)
 		if certificate.Domain == "" || certificate.Certificate == nil {
 			return errors.New("returned certificate was empty; probably an unchecked error obtaining it")
 		}