error handling when service is on down or in maintenance #28

Open
Ghost wants to merge 1 commits from <deleted>:master into master
Ghost commented 4 years ago

Hi there,

I was receiving this response for a short period of time:

{
   type: 'urn:acme:error:serverInternal',
   detail: 'The service is down for maintenance or had an internal error. Check https://letsencrypt.status.io/ for more details.'
}

As there are no error handling for this section, the error that was actually showing up before me digging on the sources was this:

Error cert_order:
Cannot read property 'termsOfService' of undefined
TypeError: Cannot read property 'termsOfService' of undefined
    at fin (/root/raftwars/source/node_modules/@root/acme/acme.js:75:23)
    at /root/raftwars/source/node_modules/@root/acme/acme.js:96:12
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
Hi there, I was receiving this response for a short period of time: ``` { type: 'urn:acme:error:serverInternal', detail: 'The service is down for maintenance or had an internal error. Check https://letsencrypt.status.io/ for more details.' } ``` As there are no error handling for this section, the error that was actually showing up before me digging on the sources was this: ``` Error cert_order: Cannot read property 'termsOfService' of undefined TypeError: Cannot read property 'termsOfService' of undefined at fin (/root/raftwars/source/node_modules/@root/acme/acme.js:75:23) at /root/raftwars/source/node_modules/@root/acme/acme.js:96:12 at processTicksAndRejections (internal/process/task_queues.js:93:5) ```
This pull request has changes conflicting with the target branch.
acme.js
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.