noop on async getCertificates error #2

Samengevoegd
coolaj86 heeft 1 commits samengevoegd van :master naar master 2019-04-15 17:36:25 +00:00
First-time contributor

This should fix #1. Otherwise, I'm happy to make a patch for other suggestions.

This should fix #1. Otherwise, I'm happy to make a patch for other suggestions.
Eigenaar

Thanks! I’ll merge this in today.

What if we put a log in there too? Is there a case where you think that would be helpful or is it just noise?

Thanks! I’ll merge this in today. What if we put a log in there too? Is there a case where you think that would be helpful or is it just noise?
Author
First-time contributor

Yeah, I think it can be noisy for end user, but helpful for dev. Adding it now.

Yeah, I think it can be noisy for end user, but helpful for dev. Adding it now.
Author
First-time contributor

Hi, this is still pending. Could you take a look at this? I've added the logging to the patch.

Hi, this is still pending. Could you take a look at this? I've added the logging to the patch.
Eigenaar

Thanks for the follow up. I must have gotten distracted. I've got this back in the queue for today.

Thanks for the follow up. I must have gotten distracted. I've got this back in the queue for today.
coolaj86 heeft deze pull request gesloten 2019-04-15 17:36:25 +00:00
Eigenaar

Published le-sni-auto@2.1.9

Published le-sni-auto@2.1.9
Log in om deel te nemen aan deze discussie.
Geen beoordelaars
Geen label
Geen mijlpaal
Niet toegewezen
2 deelnemers
Notificaties
Vervaldatum
De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Referentie: coolaj86/le-sni-auto.js#2
No description provided.