ceramicnetwork / js-ceramic

Typescript implementation of the Ceramic protocol
http://ceramic.network
Other
414 stars 127 forks source link

fix(ceramic-node): handle case where the admin DID is not authenticated #3248

Closed gvelez17 closed 2 months ago

gvelez17 commented 2 months ago

Description

When running with simple deploy, we receive the error

Error: DID is not authenticated
    at get id [as id]

handle this case and report the error, but do not fail to start

How Has This Been Tested?

Describe the tests that you ran to verify your changes. Provide instructions for reproduction.

PR checklist

Before submitting this PR, please make sure:

References:

Please list relevant documentation (e.g. tech specs, articles, related work etc.) relevant to this change, and note if the documentation has been updated.

gvelez17 commented 2 months ago

huh i thought i saw where @stbrody you wanted a warning about not being authenticated, so i changed it to add the warning - may merge on prior approvals tho since i want to get this in