Skip to content
This repository was archived by the owner on Dec 30, 2020. It is now read-only.

Scrypt call is executed in background#5

Open
abenmrad wants to merge 4 commits intoCrypho:masterfrom
LockateMe:master
Open

Scrypt call is executed in background#5
abenmrad wants to merge 4 commits intoCrypho:masterfrom
LockateMe:master

Conversation

@abenmrad
Copy link

Hi. Thanks for the work put on this module!

I would like to suggest the following modifications hereby attached:

  • scrypt call on iOS is ran in a background thread, like it's already the case on Android
  • more verifications of the provided parameters in JavaScript

@abenmrad abenmrad changed the title Scrypt call is executed in background Scrypt call is executed in background Aug 23, 2015
@ggozad
Copy link
Contributor

ggozad commented Aug 26, 2015

Ops, sorry for the delay and thanks for your contribution. Will review asap!

@ggozad
Copy link
Contributor

ggozad commented Aug 27, 2015

Hey,
I tried to run the tests, and they fail. Could you please have a look and let me know?

@abenmrad
Copy link
Author

I tested it from a script I wrote inside the app I'm working on, using some of the vectors in your test script. I'm gonna try yours.

What are you using to run your test script? I sense JasmineJS. Is there anything else? RequireJS?

@ggozad
Copy link
Contributor

ggozad commented Aug 28, 2015

The tests are contained in the plugin!
You basically create a cordova app, and add the cordova-plugin-test-framework which is the standard way of writing tests for cordova. See https://github.com/apache/cordova-plugin-test-framework for instructions.

I imagine it will be something minor, but would appreciate it if you find it and fix it, it's a good opportunity to learn the test framework too :)
While we are at it, add your name and update the Changelog!

@ggozad
Copy link
Contributor

ggozad commented Sep 2, 2015

I updated the README, with specific instructions on how you can test. Please try it and let me know.

@abenmrad
Copy link
Author

abenmrad commented Sep 3, 2015

Hi. Sorry for the late reply. And thanks for putting together the testing instructions, it helped me out and gained me time :)

I tried it out. And indeed it doesn't give the same results. It's quite intriguing. Some vectors are passing and others are not. So yeah, obviously my pull request shouldn't be merged until we figure out (and fix) what's going wrong... Thanks for pointing this out.

@ggozad
Copy link
Contributor

ggozad commented Sep 3, 2015

No problem, I will be happy to merge once you resolve these problems.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants