Merge pull request #4 from robdavid/bugfix/digest

Bugfix/digest. Thanks to @robdavid
naqvis · Mar 21, 2021 · 8635a0a · 8635a0a
2 parents 31ca429 + e53b12c
commit 8635a0a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -77,6 +77,7 @@ config = XMPP::Config.new(
   sasl_auth_order: [XMPP::AuthMechanism::SCRAM_SHA_512, XMPP::AuthMechanism::SCRAM_SHA_256,
                     XMPP::AuthMechanism::SCRAM_SHA_1, XMPP::AuthMechanism::DIGEST_MD5,
                     XMPP::AuthMechanism::PLAIN, XMPP::AuthMechanism::ANONYMOUS]
+)
 
 router = XMPP::Router.new
 

diff --git a/src/xmpp/auth/scram-sha.cr b/src/xmpp/auth/scram-sha.cr
@@ -77,7 +77,7 @@ module XMPP
 
       # stored_key = OpenSSL::SHA1.hash(client_key.to_unsafe, LibC::SizeT.new(client_key.bytesize))
       hasher.update(client_key)
-      stored_key = hasher.digest
+      stored_key = hasher.final
 
       auth_msg = "#{initial_msg},#{server_resp},#{bare_msg}"
       client_sig = OpenSSL::HMAC.digest(algorithm: algorithm, key: stored_key, data: auth_msg)