incorrect code in ChartProtection & test

Issue #764 resolved
qrilka
created an issue

openpyxl/chartsheet/tests/test_protection.py specifies SHA-512 and spinCount="100000" but hash is calculated using 1 run of sha256 and also tested hashValue is hexadecimal when it should be base64

Comments (5)

  1. qrilka reporter

    Sure but what's the point of providing incorrect implementation and adding tests for it? It's at least misleading library users.

  2. CharlieC

    Excuse me for breathing! It was a simple mistake, though I personally didn't actually write this code. Given that the algorithms for the various encryption schemes are not part of the specification then this kind of mistake is easy to make.

    Instead of complaining how about submitting a PR that fixes this?

  3. qrilka reporter

    Excuse me for bug reporting, I've found that code in Google, I wish I could have time to fix libraries I don't use. Sorry for letting you know. Feel free to close the ticket

  4. Log in to comment