Well that. At the admin page, after I have enabled the api keys, when've trying any service it have always failed.

This was because when generating the hash of the form, the hash_hmac's data parameter wasn't built using ";".

Attached is the diff patch.

Cheers!

@@ -126,7 +126,7 @@
     switch ($arg['#name']) {
  
       case 'hash':
-        $form['arg'][$key] = array('#title' => 'Hash', '#type' => 'textfield', '#default_value' => hash_hmac("sha256", $timestamp . $_SERVER['HTTP_HOST'] . $nonce . arg(4), services_admin_browse_get_first_key()));
+        $form['arg'][$key] = array('#title' => 'Hash', '#type' => 'textfield', '#default_value' => hash_hmac("sha256", $timestamp .';'. $_SERVER['HTTP_HOST'] .';'. $nonce .';'. arg(4), services_admin_browse_get_first_key()));
         break;
       
       case 'sessid':
CommentFileSizeAuthor
services_admin_browse.inc_.diff816 bytesbrutuscat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marcingy’s picture

Status: Patch (to be ported) » Needs review
kylebrowning’s picture

Version: 5.x-0.92 » 6.x-2.x-dev
Status: Needs review » Fixed

This has been fixed in 6.x-2.x-dev

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.