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

Extend Wallets to include an AuthSig field #11

Open
anthonyra opened this issue Feb 7, 2023 · 0 comments
Open

Extend Wallets to include an AuthSig field #11

anthonyra opened this issue Feb 7, 2023 · 0 comments

Comments

@anthonyra
Copy link

anthonyra commented Feb 7, 2023

I'm interested in the more flexible extension listed at #3 however for my current use case. I'd like the ability to save an AuthSig generated from the Lit Protocol to the ArProfile created with arweave-account. I'm thinking that since this is protocol specific that a JSON extension could be used to implement such a feature? I do like the flexibility of appData but don't think it's needed to implement this part. I would suggest the following change to both T_profile and ArAccountEncoded.

type T_profile | ArAccountEncoded = {
  ...
  wallets: {
    eth?: {
      address: string;
      auth?: {
        litAuthSig?: { // IAW https://lit-protocol.github.io/lit-js-sdk/api_docs_html/#authsig
          sig: string;
          derivedVia: string;
          signedMessage: string;
          address: string;
        }
      }
    }
  }
};

I believe the type upgrade above is all that's needed to implement this additional data field. Let me know if this isn't the case?

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

No branches or pull requests

1 participant