add "verify_passwd" and "verify_integrity" option

[jeffli678]

#45

[codecov-io]

Codecov Report

Merging #46 into master will decrease coverage by 1.93%.
The diff coverage is 55.4%.

@@            Coverage Diff            @@
##           master     #46      +/-   ##
=========================================
- Coverage   86.64%   84.7%   -1.94%     
=========================================
  Files          13      13              
  Lines        1168    1223      +55     
=========================================
+ Hits         1012    1036      +24     
- Misses        156     187      +31

Impacted Files	Coverage Δ
msoffcrypto/method/ecma376_standard.py	98.21% <100%> (ø)	⬆️
msoffcrypto/method/ecma376_agile.py	68.13% <51.92%> (-21.87%)	⬇️
msoffcrypto/format/ooxml.py	75.22% <57.89%> (-3.57%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36d1d0e...8785514. Read the comment docs.

[nolze]

Thanks! Overall it looks good to me. I'll take time to review and suggest some minor changes within 1-2 weeks.
To put your name in the contributors page, please make sure that your commits are associated with your GitHub account.

[nolze]

Sorry for taking time to look into this; please wait a little while.

[jeffli678]

Sorry for taking time to look into this; please wait a little while.

Never mind, please take your time!

[codecov-commenter]

Codecov Report

Merging #46 into master will increase coverage by 0.36%.
The diff coverage is 94.52%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   86.64%   87.00%   +0.36%     
==========================================
  Files          13       13              
  Lines        1168     1224      +56     
==========================================
+ Hits         1012     1065      +53     
- Misses        156      159       +3     

Impacted Files	Coverage Δ
msoffcrypto/method/rc4_cryptoapi.py	95.23% <ø> (ø)
msoffcrypto/format/ooxml.py	79.64% <84.21%> (+0.85%)	⬆️
msoffcrypto/method/ecma376_agile.py	94.38% <98.07%> (+4.38%)	⬆️
msoffcrypto/method/ecma376_standard.py	98.18% <100.00%> (-0.04%)	⬇️
msoffcrypto/format/ppt97.py	98.62% <0.00%> (-0.27%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36d1d0e...76e7700. Read the comment docs.

[nolze]

Finally I'm happy to get ready to merge your PR.

I added some changes, especially to ecma376_agile.py. Please let me know if you find any problem. Some notes:

• Did some rename
• To keep crypto methods stateless, I decided not to share the result of hashing iteration in Agile between makekey_from_password and verify_password, but do once for each (this could be improved in future)
• verify_password and verify_integrity is False by default for now, but could be True by default in 5.x. (with optimization)
• Added some tests and looks fine

EDIT: To put your name in the contributors page, please match the email address added in your GitHub account and ones used in your commits.

[jeffli678]

Your changes look nice to me. I will also check my email and let you know.

[jeffli678]

I updated my email to the correct one. If you do not have other issues, please accept the PR.

To keep track of what I did:

1. pull the remote to my local and resolve the merge conflicts. This brings you changes to my local.
2. rebase -i, and then amend the email in my two commits.
3. force push

I hope that I did it correctly. Interestingly GitHub updates the PR automatically so I do not need to file a new one.

[nolze]

Looks good to me. Thank you for your patience and cooperation!