Fix: header-based versioning with use of prefix.

CHANGELOG.markdown

@@ -11,6 +11,7 @@
 * [#302](/~https://github.com/intridea/grape/pull/302): Fix: removed redundant `autoload` entries - [@ugisozols](/~https://github.com/ugisozols).
 * [#172](/~https://github.com/intridea/grape/issues/172): Fix: MultiJson deprecated methods warnings - [@dblock](/~https://github.com/dblock).
 * [#293](/~https://github.com/intridea/grape/pull/293): Added options to `cookies.delete`, enables passing a path - [@inst](/~https://github.com/inst).
+* [#133](/~https://github.com/intridea/grape/issues/133): Fix: header-based versioning with use of `prefix` - [@seanmoon](/~https://github.com/seanmoon), [@dblock](/~https://github.com/dblock).
 * Your contribution here.
 
 0.2.3 (24/12/2012)

lib/grape/endpoint.rb

@@ -396,7 +396,7 @@ def build_middleware
 
       b.use Rack::Auth::Basic, settings[:auth][:realm], &settings[:auth][:proc] if settings[:auth] && settings[:auth][:type] == :http_basic
       b.use Rack::Auth::Digest::MD5, settings[:auth][:realm], settings[:auth][:opaque], &settings[:auth][:proc] if settings[:auth] && settings[:auth][:type] == :http_digest
-      b.use Grape::Middleware::Prefixer, :prefix => settings[:root_prefix] if settings[:root_prefix]
+      b.use Grape::Middleware::Prefixer, :prefix => settings[:root_prefix] if settings[:root_prefix] && settings[:version] && settings[:version_options][:using] == :path
 
       if settings[:version]
         b.use Grape::Middleware::Versioner.using(settings[:version_options][:using]), {

spec/grape/api_spec.rb

@@ -45,7 +45,6 @@ def app; subject end
       end
     end
   end
-
   describe '.version using param' do
     it_should_behave_like 'versioning' do
       let(:macro_options) do

spec/shared/versioning_examples.rb

@@ -79,7 +79,7 @@
     end
 
     context 'with a prefix' do
-      it 'allows the same endpoint to be implemented' do
+      it 'allows the same endpoint to be implemented' do        
         subject.prefix 'api'
         subject.version 'v2', macro_options
         subject.get 'version' do
@@ -92,12 +92,13 @@
           end
         end
 
-        versioned_get '/api/version', 'v2', macro_options
-        last_response.status.should == 200
-        last_response.body.should == 'v2'
-        versioned_get '/api/version', 'v1', macro_options
+        versioned_get '/version', 'v1', macro_options.merge(:prefix => subject.prefix)
         last_response.status.should == 200
         last_response.body.should == 'version v1'
+
+        versioned_get '/version', 'v2', macro_options.merge(:prefix => subject.prefix)
+        last_response.status.should == 200
+        last_response.body.should == 'v2'
       end
     end
   end