Fix verifySignature timestamp units - Githubissues

cloudinary / cloudinary_java

Cloudinary Java Client Library

MIT License

163 stars 110 forks source link

Fix verifySignature timestamp units #264

Closed daplf closed 2 years ago

daplf commented 2 years ago

Brief Summary of Changes

Fixes verifySignature to assume the timestamp in seconds, rather than milliseconds.

What does this PR address?

https://github.com/cloudinary/cloudinary_java/issues/260

Are tests included?

[ ] Yes
[x] No

Reviewer, please note:

The CI build is failing, not really sure why since it seems unrelated, but let me know if it's not.

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I ran the full test suite before pushing the changes and all the tests pass.