-
Notifications
You must be signed in to change notification settings - Fork 22
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
[WIP] API Rework #554
[WIP] API Rework #554
Conversation
Co-authored-by: Piotr Roslaniec <[email protected]>
Modify some default values used for creating SIWE message, including use a random nonce.
…used tests to fail. Co-authored-by: Manuel Montenegro <[email protected]>
Fix the URI used for EIP4361 message to be properly formed - it caused an invalid SIWE message to be created.
…are invalid for SIWE.
Run linter.
…61 to be more accurate and in line with EIP712 auth signature call. This can perhaps be an interface/base class method in the future. Run linter.
…rt generic auth-provider
@@ -202,6 +185,7 @@ export const isAuthorized = async ( | |||
messageKit, | |||
); | |||
|
|||
// TODO is this still valid and actually needed? should we remove this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's worthwhile to revisit it post release of @nucypher/contracts
- IIRC, there were some changes to EncryptorAllowlist
contract, etc.
So we may want to revisit how (if) to expose this logic in taco-web
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, please tag me for a re-review whenever
…et the ritual so early; obtaining it later saves us from passing additional parameters.
Allow Context to be created with a condition, and subsequently populated with authProviders and customParameters. Make :userAddressExternalEIP4361 a reserved context variable because the context will use the provided authProviders to populate its (and :userAddress) values in the overall context used for decryption.
… replaces authProvider and customParmeters since those are now encompassed in the context itself.
…common elements for both ei4361 provider and single sign-on provider. Move user address context variable for external eip4361 to taco-auth as well. Update relevant imports.
…s the only thing using it.
…d of directly passing auth provider. Expose USER_ADDRESS_PARAM* constants from taco for now.
…viding auth provider.
…roviding authProvider.
Type of PR:
Required reviews:
What this does:
Based over #547
ConditionContext
instead of splitting up params for authProvider and customParameters.ConditionContext
to be populated by developers with authProviders and customParameters before being used for decryptionIssues fixed/closed:
Why it's needed:
Notes for reviewers: