-
Notifications
You must be signed in to change notification settings - Fork 0
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
Experimental implementation of V002 proposal #7
Comments
From initial implementation. Before V002 test suite run time under
With V002 and 210K iterations instead of 10K, about 1.6 secs:
|
Proposal created/opened gsantner/markor#2471 |
Implement re-encrypt tool ptrecrypt #170 Can be used to bulk/batch/mass convert or update password for files. Operates on files (not notes/text). Also fixes v001 support for jenc, #169 * code for #170 - ptrecrypt batch conversion/re-encrypt * ptrecrypt and experimental v002 jenc support #170 And clach04/jenc-py#7
similar comment posted to Markor discussion: I still do not have an android build (even though, I've written the code 😆). What I have done is implement some testing with a desktop/command-line, python implementation. This implementation is single-threaded, it does NOT attempt to do anything in parallel. I ran the tests at least 3 times ('cos this is supposed to be a fun project). I have an SSD and I will ignore the first run so as to not test disk-IO timing, and instead focus on the CPU processing time. I have essentially done 2 benchmarks with 4 different encryption implementations (2 jenc).
The benchmark searches both the plain text and the encrypted files so one can argue that case 2 above is not solely testing crypto impact, but it matches my real use cases :-) Stats for 2:
Results - 1Results 1 - Raw
(py3.12.5venv) C:\code\puren_tonbo>ptgrep -p password --time --search_encrypted does_not_exist C:\code\notes\10000-markdown-files\10k_v001_jenc
Results 1 - Summary
Results - 2Results 2 - Raw
(py3.12.5venv) C:\tmp\work_notes>ptgrep --time --search_encrypted really_does_not_exist tombo
Results 1 - Summary
Background NotesI used a new feature in puren_tonbo to generate the test data, there is a new tool for bulk converting files: For 10K notes:
For my notes: Copy tombo folder a few times and name using dirs below, re-encrypt in-place:
|
Experimental implementation of V002 proposal. Available in https://github.com/clach04/jenc-py/tree/issue_7_V002_experiment
The same as V001 but adopt iteration count recommendation from OWASP. I.e. 210K taken 2024-11-12 from https://cheatsheetseries.owasp.org/cheatsheets/Password_Storage_Cheat_Sheet.html#pbkdf2
As per comment gsantner/markor#1378 (comment)
Discussion on Markor gsantner/markor#2471
The text was updated successfully, but these errors were encountered: