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

已关闭
由 Ghost 于 2019-04-04 19:59:33 +00:00 打开 · 6 评论

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!
管理员

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.
管理员

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.
作者

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
管理员

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.
管理员

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`).
作者

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!
Ghost2019-04-05 16:37:35 +00:00 关闭此工单
登录 并参与到对话中。
未选择标签
未选择里程碑
未指派成员
2 名参与者
通知
到期时间
到期日期无效或超出范围。请使用 'yyyy-mm-dd' 格式。

未设置到期时间。

依赖工单

没有设置依赖项。

参考:coolaj86/acme.js-ARCHIVED#19
没有提供说明。