On 08/17/11 - 09:57:56AM, Chris Lalancette wrote:
> Hey Marios,
>      I know this is several months out of date, but I was just doing some
> testing on the blob creation stuff and noticing that my libdeltacloud tests
> were failing.  I traced it down to the fact that the blob_id parameter changed
> from param[:blob_id] to param[:blob] when you added the streaming stuff to
> blobs.
>      If I remember correctly, deltacloud 0.3.0 was released with bucket/blob
> stuff (even if it wasn't streaming), so I think we'll probably want to
> re-instate :blob_id as a parameter to keep backwards compatibility.  What do
> you think?

Maybe like this patch?

-- 
Chris Lalancette
>From bd975453f58ac7f5da2fc51d75a3535a24d0da24 Mon Sep 17 00:00:00 2001
From: Chris Lalancette <[email protected]>
Date: Wed, 17 Aug 2011 10:02:07 -0400
Subject: [PATCH] Allow params['blob_id'] for backwards compatibility.

Signed-off-by: Chris Lalancette <[email protected]>
---
 server/server.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/server/server.rb b/server/server.rb
index 2355278..6bdb066 100644
--- a/server/server.rb
+++ b/server/server.rb
@@ -779,7 +779,7 @@ end
 #create a new blob using html interface - NON STREAMING (i.e. browser POST 
http form data)
 post "#{Sinatra::UrlForHelper::DEFAULT_URI_PREFIX}/buckets/:bucket" do
   bucket_id = params[:bucket]
-  blob_id = params['blob']
+  blob_id = params['blob'] || params['blob_id']
   blob_data = params['blob_data']
   user_meta = {}
   #metadata from params (i.e., passed by http form post, e.g. browser)
-- 
1.7.4.4

Reply via email to