Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: print provision messages when auto-provision is triggered #3077

Merged
merged 2 commits into from
Jul 2, 2024

Conversation

TheTechmage
Copy link
Contributor

It was recently discovered that if you set a static seed, and you have auto-provisioning enabled on the start command, there is no way to get the verkey for the created DID. The verkey is never printed out to the command line and I do not remember there being an API for retrieving the verkey (one of those, "See it once, copy it, or forever lose it" kinds of things).

After this point in the code, the only thing that "provision" enables are the print statements talking about the did/verkey being created as well as some basic profile information. It doesn't trigger any code that would make changes to the system, those lines precede the line I've added.

It was recently discovered that if you set a static seed, and you have
auto-provisioning enabled on the start command, there is no way to get
the verkey for the created DID. The verkey is never printed out to the
command line and I do not remember there being an API for retrieving the
verkey (one of those, "See it once, copy it, or forever lose it" kinds
of things).

After this point in the code, the only thing that "provision" enables
are the print statements talking about the did/verkey being created as
well as some basic profile information. It doesn't trigger any code that
would make changes to the system, those lines precede the line I've
added.

Signed-off-by: Colton Wolkins (Indicio work address) <[email protected]>
@jamshale jamshale enabled auto-merge (squash) July 2, 2024 17:09
@jamshale jamshale merged commit 559b2f9 into openwallet-foundation:main Jul 2, 2024
7 checks passed
Copy link

sonarqubecloud bot commented Jul 2, 2024

@TheTechmage TheTechmage deleted the fix/provision branch July 2, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants