This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Fix hmac bindings #21

Open

villesau wants to merge 1 commit into reasonml-old:master from villesau:master

villesau commented Dec 3, 2019 •

edited

Loading

Previously it was not possible to do something like this:

Crypto.Hmac.(create(`SHA256, hmacKey)->appendString(str)->toHex)

which is totally valid.

Hmac node docs: https://nodejs.org/api/crypto.html#crypto_hmac_update_data_inputencoding
Documentation for allowed encodings (implemented only hex this time): https://nodejs.org/api/buffer.html#buffer_buffers_and_character_encodings


          fix hmac bindings

08ac3ce

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

Labels

None yet