processone / ejabberd-contrib

Growing and curated ejabberd contributions repository - PR or ask to join !
http://ejabberd.im
248 stars 137 forks source link

Installation of ejabberd_auth_http throws warning "call to crypto:rand_bytes" #243

Closed m-byte closed 5 years ago

m-byte commented 6 years ago

When running docker exec -it ejabberd bin/ejabberdctl module_install ejabberd_auth_http on ejabberd/ecs, the following warning gets thrown:

/home/ejabberd/.ejabberd-modules/sources/ejabberd-contrib/ejabberd_auth_http/src/scram2.erl:135:
Warning: call to crypto:rand_bytes/1 will fail, since it was removed in 20.0; use crypto:strong_rand_bytes/1

The line causing this: https://github.com/processone/ejabberd-contrib/blob/721ca125ea0b8f5d16c2855b114eb6715cfad9d4/ejabberd_auth_http/src/scram2.erl#L135

badlop commented 5 years ago

I've applied the fix, updated to the current prefered function.