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.

Sign up for GitHub

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

Jump to bottom

feat: implement ceramic session #246

Merged
merged 11 commits into from
Jun 19, 2023
Merged

feat: implement ceramic session #246

merged 11 commits into from
Jun 19, 2023

Conversation

andyv09
Copy link
Member

@andyv09 andyv09 commented Jun 15, 2023 •
edited
Loading

Masca:

  • Added ceramicSessionKey to account state
  • Added 2 new RPC methods; setCeramicSessionKey and verifyStoredCeramicSessionKey
  • Changed Ceramic Client to start using session from state

Connector:

  • Added new RPC methods to API
  • Added function that checks if there's a valid sessionKey in Masca. If there isn't one it tries to create one using personal_sign.
  • Added checks to each function that uses ceramic

Types:

  • Added new types

martines3000
martines3000 requested changes Jun 15, 2023
View reviewed changes
packages/connector/src/utils.ts Show resolved Hide resolved
packages/connector/src/utils.ts Outdated Show resolved Hide resolved
packages/connector/src/utils.ts Outdated Show resolved Hide resolved
packages/connector/src/utils.ts Outdated Show resolved Hide resolved
packages/snap/src/rpc/snap/ceramic.ts Outdated Show resolved Hide resolved
packages/snap/src/utils/ceramicUtils.ts Outdated Show resolved Hide resolved
packages/snap/src/utils/ceramicUtils.ts Outdated Show resolved Hide resolved
packages/connector/src/snap.ts Outdated Show resolved Hide resolved
packages/connector/package.json Show resolved Hide resolved
packages/connector/src/snap.ts Outdated Show resolved Hide resolved
@martines3000 martines3000 merged commit dc6f985 into develop Jun 19, 2023
@martines3000 martines3000 deleted the feat/ceramic-session branch June 19, 2023 08:44
@martines3000 martines3000 mentioned this pull request Jun 19, 2023
Closed
This was referenced Jun 19, 2023
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pseudobun pseudobun Awaiting requested review from pseudobun

@martines3000 martines3000 Awaiting requested review from martines3000

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andyv09 @martines3000