ERROR: TypeError: Cannot read property 'kid' of undefined #19

zavřený
otevřeno 2019-04-04 19:59:33 +00:00 uživatelem Ghost · 6 komentářů

I have encountered the following error when running the script to get a new certificate:

ERROR: TypeError: Cannot read property 'kid' of undefined

After a little bit of debugging it seems the error is raised when indexing into the undefined account object in this line, which might be due to the breaking changes mentioned here.

I was able to overcome my issue by making the following monkey patch to the line

if (options.accountKid || options.account && options.account.kid) {

Looking to see if I should submit a pull request or if a maintainer wants to push this update or any advice on the issue is very much appreciated.

Thanks!

I have encountered the following error when running the script to get a new certificate: `ERROR: TypeError: Cannot read property 'kid' of undefined` After a little bit of debugging it seems the error is raised when indexing into the undefined account object in this [line](https://git.coolaj86.com/coolaj86/acme-v2.js/src/branch/master/node.js#L701), which might be due to the breaking changes mentioned [here](https://git.coolaj86.com/coolaj86/acme-v2.js/issues/8). I was able to overcome my issue by making the following monkey patch to the line `if (options.accountKid || options.account && options.account.kid) {` Looking to see if I should submit a pull request or if a maintainer wants to push this update or any advice on the issue is very much appreciated. Thanks!
Vlastník

Thanks for the report. I'll fix that

You could tell me which plugins you're using and which versions? And what "script"?

For example:

  • greenlock@v2.7.6
  • le-challenge-dns@v3.0.2

That upcoming breaking change is unrelated to the current code (and won't break the code).

It shouldn't be possible for something to get to this module without an account in the first place... but I'll push that fix momentarily anyway. Again, thank you.

Thanks for the report. I'll fix that You could tell me which plugins you're using and which versions? And what "script"? For example: * greenlock@v2.7.6 * le-challenge-dns@v3.0.2 That upcoming breaking change is unrelated to the current code (and won't break the code). It _shouldn't_ be possible for something to get to this module without an account in the first place... but I'll push that fix momentarily anyway. Again, thank you.
Vlastník

Published as v1.7.5. But please do let me know the conditions that were broke it so I can get that fixed as well.

Published as v1.7.5. But please do let me know the conditions that were broke it so I can get that fixed as well.
Autor

I am using greenlock@v2.7.6 and le-challenge-dns@v2.3.2

Not a problem at all, thank you for responding and maintaining this repository and let me know if I can provide anything else to help

I am using greenlock@v2.7.6 and le-challenge-dns@v2.3.2 Not a problem at all, thank you for responding and maintaining this repository and let me know if I can provide anything else to help
Vlastník

Hmm... those two versions are a bit out of sync.

You'll need to have le-challenge-dns@v3.0.2, which should be in the package-lock.json for greenlock@2.7.6. But that's unrelated, I would think.

I definitely saw the code where, during account registration, the account object is creating if it doesn't exist. I'm curious as to how that error is happening.

Anyway, do an npm update; npm install and see if you still get the error.

Hmm... those two versions are a bit out of sync. You'll need to have `le-challenge-dns@v3.0.2`, which should be in the package-lock.json for greenlock@2.7.6. But that's unrelated, I would think. I definitely saw the code where, during account registration, the account object is creating if it doesn't exist. I'm curious as to how that error is happening. Anyway, do an `npm update; npm install` and see if you still get the error.
Vlastník

If you have a package-lock.json in your project, maybe remove that first so it doesn't prevent updates (though it shouldn't block an explicit npm update).

If you have a `package-lock.json` in your project, maybe remove that first so it doesn't prevent updates (though it shouldn't block an explicit `npm update`).
Autor

I've done an update which included your patch fix and it seems to have solved my error 👍

thanks again for the help!

I've done an update which included your patch fix and it seems to have solved my error :+1: thanks again for the help!
Ghost uzavřel/a tento úkol 2019-04-05 16:37:35 +00:00
Přihlaste se pro zapojení do konverzace.
Bez štítku
Bez milníku
Bez zpracovatelů
2 účastníků
Oznámení
Termín dokončení
Termín dokončení není platný nebo je mimo rozsah. Použijte prosím formát „rrrr-mm-dd“.

Žádný termín dokončení.

Závislosti

Nejsou nastaveny žádné závislosti.

Reference: coolaj86/acme.js-ARCHIVED#19
No description provided.