fixed the bug (#10)

BinaryStorms · Nov 26, 2018 · 5a615cb · 5a615cb
1 parent ea5175c
commit 5a615cb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/civic_sip_sdk/client.rb b/lib/civic_sip_sdk/client.rb
@@ -16,7 +16,7 @@ class Client
     ENV_VAR = 'CIVIC_SIP_SDK_ENV'
     TEST_ENV = 'test'
 
-    HTTP_REQUEST_METHOD = :POST
+    HTTP_REQUEST_METHOD = 'POST'
 
     # Creates a client
     #
@@ -40,7 +40,7 @@ def exchange_code(jwt_token:)
           'Content-Type' => MIMETYPE_JSON,
           'Accept' => MIMETYPE_JSON,
           'Content-Length' => json_body_str.size.to_s,
-          'Authorization' => authorization_header(target_path: AUTH_CODE_PATH, body: json_body_str)
+          'Authorization' => authorization_header(body: json_body_str)
         },
         body: json_body_str
       )
@@ -57,13 +57,13 @@ def exchange_code(jwt_token:)
 
     private
 
-    def authorization_header(target_path:, body:)
+    def authorization_header(body:)
       jwt_token = Crypto.jwt_token(
         app_id: @config.id,
         sip_base_url: BASE_URL,
         data: {
           method: HTTP_REQUEST_METHOD,
-          path: target_path
+          path: AUTH_CODE_PATH
         },
         private_key: @config.private_key
       )

diff --git a/lib/civic_sip_sdk/crypto.rb b/lib/civic_sip_sdk/crypto.rb
@@ -72,7 +72,7 @@ def self.decode_jwt_token(token:, public_hex_key:, should_verify:)
     end
 
     def self.civic_extension(secret:, body:)
-      hmac = OpenSSL::HMAC.digest(HMAC_DIGEST, secret, JSON.generate(body))
+      hmac = OpenSSL::HMAC.digest(HMAC_DIGEST, secret, body)
       Base64.encode64(hmac)
     end