-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Operation request: RSA operations #272
Comments
Started work under the rsa branch, current progress:
|
@GCHQ77703 EDIT: I've managed to finally make it import with a hacky solution using Another edit: Looks like we've run around into the OperationConfig issue again, which is the actual reason for the |
@artemisbot Do these issues not occur in the Although I must admit to having never used workers before for anything more than caching, some of the operations were taking 5-10 seconds of time with no degradation of the UI, which implies it is successfully being sent off to another worker thread. |
I tried using your commit on your fork with `grunt dev` and was just getting an error in the webapp about forge's workers not loading.
|
Ooh, interesting. It worked fine in Chrome/Chromium when I just tested it but doesn't in Firefox which is what I use normally. Wonder what's causing that? (this is on exactly the same dev build and machine) |
Ah, true. Can reproduce the problem in Firefox. Might indeed need to wait on merging this. |
Looks like digitalbazaar/forge#645 merged, I've just checked and it seems to be working totally fine with the previous branch. Going to clean up the operation and create a PR. |
Thanks for all your work. This has now been implemented and merged in #652 |
Summary
Various RSA-related operations could be added. As we are already importing the forge library, it would make sense to utilise it for this.
Possible operations:
The text was updated successfully, but these errors were encountered: