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

feat: using internet identity service #90

Merged
merged 12 commits into from
May 10, 2021
Merged

Conversation

krpeacock
Copy link
Contributor

will resolve #79

@krpeacock
Copy link
Contributor Author

krpeacock commented Apr 30, 2021

Note - works up to the point where we try to add the principal via dfx using https://identity.messaging.ic0.app/authorize

Also, this new authentication flow also prevents the dashboard from rendering preemptively #67

@krpeacock krpeacock marked this pull request as draft April 30, 2021 21:05
@krpeacock krpeacock force-pushed the kyle/internet-identity branch from 99690da to 0f318be Compare April 30, 2021 21:07
@krpeacock
Copy link
Contributor Author

Unfortunately, even with dfx 0.7.0-beta.5 and running the identity service locally, I'm still getting the error that the signature is rejected.

  Code: 403 (Forbidden)
  Body: Failed to authenticate request 0xea3481cde19d18cf9e35ec32efabd429c8a8ab5eba62d191286c2d5ee99e4481 due to: Invalid delegation: Invalid canister signature: Malformed IcCanisterSignature signature: [2107210724071602100799] error: 'signature CBOR doesn't have a self-describing tag'

    at HttpAgent.query (index.js:166)
    at async caller (actor.js:118)
    at async Object.name (index.ts:113)```

@krpeacock krpeacock marked this pull request as ready for review May 4, 2021 23:33
@krpeacock krpeacock requested a review from hansl May 5, 2021 00:30
@krpeacock
Copy link
Contributor Author

My one concern about this is that no development wallet UI will work with the production Identity Service. We still need a strategy for local development

@krpeacock krpeacock merged commit 06bb256 into next May 10, 2021
@krpeacock krpeacock deleted the kyle/internet-identity branch May 10, 2021 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants