You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@deltacloud.apache.org by Francesco Vollero <ra...@gmail.com> on 2010/12/10 13:14:31 UTC
[PATCH core 1/2] On server/lib/sinatra/rabbit.rb: Fixed the variable declaration in a more ruby way On server/server.rb: Fixed on line 72 the redirect, because it's a permanent redirect and not a 302.
---
server/lib/sinatra/rabbit.rb | 2 +-
server/server.rb | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/server/lib/sinatra/rabbit.rb b/server/lib/sinatra/rabbit.rb
index 05bd6c5..e843366 100644
--- a/server/lib/sinatra/rabbit.rb
+++ b/server/lib/sinatra/rabbit.rb
@@ -72,7 +72,7 @@ module Sinatra
end
def path(args = {})
- l_prefix = args[:prefix] ? args[:prefix] : prefix
+ l_prefix = args[:prefix] || prefix
if @member
if standard?
"#{l_prefix}/#{@collection.name}/:id"
diff --git a/server/server.rb b/server/server.rb
index 9585f9b..6b446ba 100644
--- a/server/server.rb
+++ b/server/server.rb
@@ -69,7 +69,7 @@ end
Sinatra::Application.register Sinatra::RespondTo
# Redirect to /api
-get '/' do redirect url_for('/api'); end
+get '/' do redirect url_for('/api'), 301; end
get '/api/drivers\/?' do
respond_to do |format|
--
1.7.3.3
[PATCH core 2/2] Removed unuseful escaping from get
Posted by Francesco Vollero <ra...@gmail.com>.
---
server/server.rb | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/server/server.rb b/server/server.rb
index 6b446ba..3a261f7 100644
--- a/server/server.rb
+++ b/server/server.rb
@@ -71,13 +71,13 @@ Sinatra::Application.register Sinatra::RespondTo
# Redirect to /api
get '/' do redirect url_for('/api'), 301; end
-get '/api/drivers\/?' do
+get '/api/drivers/?' do
respond_to do |format|
format.xml { haml :"api/drivers" }
end
end
-get '/api\/?' do
+get '/api/?' do
if params[:force_auth]
return [401, 'Authentication failed'] unless driver.valid_credentials?(credentials)
end
--
1.7.3.3