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

[Request]: Expanded documentation for Askar WalletConfig #128

Open
3 tasks done
daidoji opened this issue Jun 26, 2023 · 2 comments
Open
3 tasks done

[Request]: Expanded documentation for Askar WalletConfig #128

daidoji opened this issue Jun 26, 2023 · 2 comments
Labels
request Request for documentation

Comments

@daidoji
Copy link
Contributor

daidoji commented Jun 26, 2023

Documentation Request

Right now there exists https://aries.js.org/guides/0.4/tutorials/agent-config#walletconfigstorage which isn't quite helpful, although there are many examples throughout the docs of path: uri: and other options in postgres for example scattered here and there.

In trying to determine whether I could use the :memory: database with sqlite for testing I found these two options along with references on google to inMemory (which didn't seem to work). This section having a robust breakdown of the various storage backends and wallets one could use with PG, sqlite, or any other backing that might come up and a strict interface would certainly help clear up some of this for newcomers.

  1. Expand this section in the linked docs to include basic storage options for sqlite (and maybe move some of the Postgres options).
  2. Document somewhere how to use the in-memory database from sqlite either here or in its own section. (In my research passing uri: 'sqlite://:memory:' as a storage config key seemed to work at initializing agents at the least. attached file from basic config example from the docs with this for testing).

minimal_memory_example.ts.txt

Version

0.4.x

Which platform are you most interested in for documentation?

  • React Native
  • Node.JS

Contribute (optional)

  • Agree
@daidoji daidoji added the request Request for documentation label Jun 26, 2023
@TimoGlastra
Copy link
Contributor

@genaris to add some pointers on the different options

@genaris
Copy link
Contributor

genaris commented Jul 5, 2023

I think current documentation for wallet configuration is still based on the structure used for Indy SDK, while there are some particular parameters that depend on each wallet implementation. This is a conflicting point when it comes to storage configuration, as the parameters used in Indy are a bit different from the ones used in Askar.

The in-memory wallet @daidoji is referring to is a particular case of sqlite configuration for Askar (not sure if Indy allows for this as well), accessible when the storage configuration object does include an inMemory field set to true (no need to set anything else in the uri field). This is only for test purposes and was not deeply explored, but after the fix in openwallet-foundation/credo-ts#1498 it should work.

I think it would be nice to reference both Askar and Indy SDK wallet implementation details in https://aries.js.org/guides/0.4/tutorials/agent-config#walletconfigstorage and maybe rework a bit the Postgres section to be not so tied to indy-sdk.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request Request for documentation
Projects
None yet
Development

No branches or pull requests

3 participants