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

Allows the user to implement the logic for identity object creation #234

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

msabansal
Copy link

Similar to whats mentioned in #232 we have a scenario where we want to use certificates which are non-exportable.
Coming up with a clean cross platform design is challenging. This is our attempt to keep things simple and delegate the complexity to the user

@sfackler
Copy link
Owner

I do not want to expose the underlying implementing crates to the user.

@msabansal
Copy link
Author

msabansal commented Sep 16, 2022

While I understand the reasoning for it designing a cross platform feature for something thats an abstraction over the underlay feels non trivial.

If it is acceptable we can hide this behind a feature flag

@sfackler
Copy link
Owner

It is absolutely nontrivial, but that is what this library is.

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

Successfully merging this pull request may close these issues.

2 participants