From d458e39e314455be3110a619cb614f30ee461445 Mon Sep 17 00:00:00 2001
From: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
Date: Thu, 12 Feb 2015 13:47:23 +0100
Subject: Don't cache classes in tests

Signed-off-by: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
---
 config/environments/test.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'config')

diff --git a/config/environments/test.rb b/config/environments/test.rb
index 25b082b98da..2d5e7addcd3 100644
--- a/config/environments/test.rb
+++ b/config/environments/test.rb
@@ -5,7 +5,7 @@ Gitlab::Application.configure do
   # test suite. You never need to work with it otherwise. Remember that
   # your test database is "scratch space" for the test suite and is wiped
   # and recreated between test runs. Don't rely on the data there!
-  config.cache_classes = true
+  config.cache_classes = false
 
   # Configure static asset server for tests with Cache-Control for performance
   config.serve_static_assets = true
-- 
cgit v1.2.1


From 76aad9b76ed756ca9ba2cbcdb399c815e542b3ae Mon Sep 17 00:00:00 2001
From: Vinnie Okada <vokada@mrvinn.com>
Date: Sat, 24 Jan 2015 11:02:58 -0700
Subject: Upgrade to Rails 4.1.9

Make the following changes to deal with new behavior in Rails 4.1.2:

* Use nested resources to avoid slashes in arguments to path helpers.
---
 config/routes.rb | 292 +++++++++++++++++++++++++++++++------------------------
 1 file changed, 167 insertions(+), 125 deletions(-)

(limited to 'config')

diff --git a/config/routes.rb b/config/routes.rb
index 65786d83566..934653a2245 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -125,9 +125,17 @@ Gitlab::Application.routes.draw do
     resource :logs, only: [:show]
     resource :background_jobs, controller: 'background_jobs', only: [:show]
 
-    resources :projects, constraints: { id: /[a-zA-Z.\/0-9_\-]+/ }, only: [:index, :show] do
-      member do
-        put :transfer
+    resources :namespaces, path: '/projects', constraints: { id: /[a-zA-Z.0-9_\-]+/ }, only: [] do
+      root to: 'projects#index', as: :projects
+
+      resources(:projects, path: '/',
+                constraints: { id: /[a-zA-Z.0-9_\-]+/ },
+                only: [:index, :show]) do
+        root to: 'projects#show'
+
+        member do
+          put :transfer
+        end
       end
     end
 
@@ -212,167 +220,203 @@ Gitlab::Application.routes.draw do
   devise_scope :user do
     get '/users/auth/:provider/omniauth_error' => 'omniauth_callbacks#omniauth_error', as: :omniauth_error
   end
+
+  root to: "dashboard#show"
+
   #
   # Project Area
   #
-  resources :projects, constraints: { id: /[a-zA-Z.0-9_\-]+\/[a-zA-Z.0-9_\-]+/ }, except: [:new, :create, :index], path: '/' do
-    member do
-      put :transfer
-      post :archive
-      post :unarchive
-      post :upload_image
-      post :toggle_star
-      post :markdown_preview
-      get :autocomplete_sources
-    end
-
-    scope module: :projects do
-      # Blob routes:
-      get '/new/:id', to: 'blob#new', constraints: { id: /.+/ }, as: 'new_blob'
-      post '/create/:id', to: 'blob#create', constraints: { id: /.+/ }, as: 'create_blob'
-      get '/edit/:id', to: 'blob#edit', constraints: { id: /.+/ }, as: 'edit_blob'
-      put '/update/:id', to: 'blob#update', constraints: { id: /.+/ }, as: 'update_blob'
-      post '/preview/:id', to: 'blob#preview', constraints: { id: /.+/ }, as: 'preview_blob'
-
-      resources :blob, only: [:show, :destroy], constraints: { id: /.+/, format: false } do
-        get :diff, on: :member
+  resources :namespaces, path: '/', constraints: { id: /[a-zA-Z.0-9_\-]+/ }, only: [] do
+    resources(:projects, constraints: { id: /[a-zA-Z.0-9_\-]+/ }, except:
+              [:new, :create, :index], path: "/") do
+      member do
+        put :transfer
+        post :archive
+        post :unarchive
+        post :upload_image
+        post :toggle_star
+        post :markdown_preview
+        get :autocomplete_sources
       end
 
-      resources :raw,       only: [:show], constraints: { id: /.+/ }
-      resources :tree,      only: [:show], constraints: { id: /.+/, format: /(html|js)/ }
-      resource  :avatar,    only: [:show, :destroy]
-
-      resources :commit,    only: [:show], constraints: { id: /[[:alnum:]]{6,40}/ } do
-        get :branches, on: :member
-      end
+      scope module: :projects do
+        # Blob routes:
+        get '/new/*id', to: 'blob#new', constraints: { id: /.+/ }, as: 'new_blob'
+        post '/create/*id', to: 'blob#create', constraints: { id: /.+/ }, as: 'create_blob'
+        get '/edit/*id', to: 'blob#edit', constraints: { id: /.+/ }, as: 'edit_blob'
+        put '/update/*id', to: 'blob#update', constraints: { id: /.+/ }, as: 'update_blob'
+        post '/preview/*id', to: 'blob#preview', constraints: { id: /.+/ }, as: 'preview_blob'
+
+        scope do
+          get('/blob/*id/diff', to: 'blob#diff',
+              constraints: { id: /.+/, format: false },
+              as: :blob_diff)
+          get('/blob/*id', to: 'blob#show',
+              constraints: { id: /.+/, format: false }, as: :blob)
+          delete('/blob/*id', to: 'blob#destroy',
+                 constraints: { id: /.+/, format: false })
+        end
 
-      resources :commits,   only: [:show], constraints: { id: /(?:[^.]|\.(?!atom$))+/, format: /atom/ }
-      resources :compare,   only: [:index, :create]
-      resources :blame,     only: [:show], constraints: { id: /.+/ }
-      resources :network,   only: [:show], constraints: { id: /(?:[^.]|\.(?!json$))+/, format: /json/ }
-      resources :graphs,    only: [:show], constraints: { id: /(?:[^.]|\.(?!json$))+/, format: /json/ } do
-        member do
-          get :commits
+        scope do
+          get(
+            '/raw/*id',
+            to: 'raw#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :raw
+          )
         end
-      end
 
-      get '/compare/:from...:to' => 'compare#show', :as => 'compare',
-          :constraints => { from: /.+/, to: /.+/ }
+        scope do
+          get(
+            '/tree/*id',
+            to: 'tree#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :tree
+          )
+        end
+        resource  :avatar,    only: [:show, :destroy]
 
-      resources :snippets, constraints: { id: /\d+/ } do
-        member do
-          get 'raw'
+        resources :commit,    only: [:show], constraints: { id: /[[:alnum:]]{6,40}/ } do
+          get :branches, on: :member
         end
-      end
 
-      resources :wikis, only: [:show, :edit, :destroy, :create], constraints: { id: /[a-zA-Z.0-9_\-\/]+/ } do
-        collection do
-          get :pages
-          put ':id' => 'wikis#update'
-          get :git_access
+        resources :commits,   only: [:show], constraints: { id: /(?:[^.]|\.(?!atom$))+/, format: /atom/ }
+        resources :compare,   only: [:index, :create]
+
+        scope do
+          get(
+            '/blame/*id',
+            to: 'blame#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :blame
+          )
         end
 
-        member do
-          get 'history'
+        resources :network,   only: [:show], constraints: { id: /(?:[^.]|\.(?!json$))+/, format: /json/ }
+        resources :graphs,    only: [:show], constraints: { id: /(?:[^.]|\.(?!json$))+/, format: /json/ } do
+          member do
+            get :commits
+          end
         end
-      end
 
-      resource :fork, only: [:new, :create]
-      resource :import, only: [:new, :create, :show]
+        get '/compare/:from...:to' => 'compare#show', :as => 'compare',
+            :constraints => { from: /.+/, to: /.+/ }
 
-      resource :repository, only: [:show, :create] do
-        member do
-          get 'archive', constraints: { format: Gitlab::Regex.archive_formats_regex }
+        resources :snippets, constraints: { id: /\d+/ } do
+          member do
+            get 'raw'
+          end
         end
-      end
 
-      resources :services, constraints: { id: /[^\/]+/ }, only: [:index, :edit, :update] do
-        member do
-          get :test
-        end
-      end
+        resources :wikis, only: [:show, :edit, :destroy, :create], constraints: { id: /[a-zA-Z.0-9_\-\/]+/ } do
+          collection do
+            get :pages
+            put ':id' => 'wikis#update'
+            get :git_access
+          end
 
-      resources :deploy_keys, constraints: { id: /\d+/ } do
-        member do
-          put :enable
-          put :disable
+          member do
+            get 'history'
+          end
         end
-      end
 
-      resources :branches, only: [:index, :new, :create, :destroy], constraints: { id: Gitlab::Regex.git_reference_regex }
-      resources :tags, only: [:index, :new, :create, :destroy], constraints: { id: Gitlab::Regex.git_reference_regex }
-      resources :protected_branches, only: [:index, :create, :update, :destroy], constraints: { id: Gitlab::Regex.git_reference_regex }
+        resource :repository, only: [:show, :create] do
+          member do
+            get 'archive', constraints: { format: Gitlab::Regex.archive_formats_regex }
+          end
+        end
 
-      resources :refs, only: [] do
-        collection do
-          get 'switch'
+        resources :services, constraints: { id: /[^\/]+/ }, only: [:index, :edit, :update] do
+          member do
+            get :test
+          end
         end
 
-        member do
-          # tree viewer logs
-          get 'logs_tree', constraints: { id: Gitlab::Regex.git_reference_regex }
-          get 'logs_tree/:path' => 'refs#logs_tree', as: :logs_file, constraints: {
-            id: Gitlab::Regex.git_reference_regex,
-            path: /.*/
-          }
+        resources :deploy_keys, constraints: { id: /\d+/ } do
+          member do
+            put :enable
+            put :disable
+          end
         end
-      end
 
-      resources :merge_requests, constraints: { id: /\d+/ }, except: [:destroy] do
-        member do
-          get :diffs
-          post :automerge
-          get :automerge_check
-          get :ci_status
+        resource :fork, only: [:new, :create]
+        resource :import, only: [:new, :create, :show]
+
+        resources :refs, only: [] do
+          collection do
+            get 'switch'
+          end
+
+          member do
+            # tree viewer logs
+            get 'logs_tree', constraints: { id: Gitlab::Regex.git_reference_regex }
+            get 'logs_tree/:path' => 'refs#logs_tree', as: :logs_file, constraints: {
+              id: Gitlab::Regex.git_reference_regex,
+              path: /.*/
+            }
+          end
         end
 
-        collection do
-          get :branch_from
-          get :branch_to
-          get :update_branches
+        resources :merge_requests, constraints: { id: /\d+/ }, except: [:destroy] do
+          member do
+            get :diffs
+            post :automerge
+            get :automerge_check
+            get :ci_status
+          end
+
+          collection do
+            get :branch_from
+            get :branch_to
+            get :update_branches
+          end
         end
-      end
 
-      resources :hooks, only: [:index, :create, :destroy], constraints: { id: /\d+/ } do
-        member do
-          get :test
+        resources :branches, only: [:index, :new, :create, :destroy], constraints: { id: Gitlab::Regex.git_reference_regex }
+        resources :tags, only: [:index, :new, :create, :destroy], constraints: { id: Gitlab::Regex.git_reference_regex }
+        resources :protected_branches, only: [:index, :create, :update, :destroy], constraints: { id: Gitlab::Regex.git_reference_regex }
+
+        resources :hooks, only: [:index, :create, :destroy], constraints: { id: /\d+/ } do
+          member do
+            get :test
+          end
         end
-      end
 
-      resources :team, controller: 'team_members', only: [:index]
-      resources :milestones, except: [:destroy], constraints: { id: /\d+/ } do
-        member do
-          put :sort_issues
-          put :sort_merge_requests
+        resources :team, controller: 'team_members', only: [:index]
+        resources :milestones, except: [:destroy], constraints: { id: /\d+/ } do
+          member do
+            put :sort_issues
+            put :sort_merge_requests
+          end
         end
-      end
 
-      resources :labels, constraints: { id: /\d+/ } do
-        collection do
-          post :generate
+        resources :labels, constraints: { id: /\d+/ } do
+          collection do
+            post :generate
+          end
         end
-      end
 
-      resources :issues, constraints: { id: /\d+/ }, except: [:destroy] do
-        collection do
-          post  :bulk_update
+        resources :issues, constraints: { id: /\d+/ }, except: [:destroy] do
+          collection do
+            post  :bulk_update
+          end
         end
-      end
 
-      resources :team_members, except: [:index, :edit], constraints: { id: /[a-zA-Z.\/0-9_\-#%+]+/ } do
-        collection do
-          delete :leave
+        resources :team_members, except: [:index, :edit], constraints: { id: /[a-zA-Z.\/0-9_\-#%+]+/ } do
+          collection do
+            delete :leave
 
-          # Used for import team
-          # from another project
-          get :import
-          post :apply_import
+            # Used for import team
+            # from another project
+            get :import
+            post :apply_import
+          end
         end
-      end
 
-      resources :notes, only: [:index, :create, :destroy, :update], constraints: { id: /\d+/ } do
-        member do
-          delete :delete_attachment
+        resources :notes, only: [:index, :create, :destroy, :update], constraints: { id: /\d+/ } do
+          member do
+            delete :delete_attachment
+          end
         end
       end
 
@@ -380,6 +424,4 @@ Gitlab::Application.routes.draw do
   end
 
   get ':id' => 'namespaces#show', constraints: { id: /(?:[^.]|\.(?!atom$))+/, format: /atom/ }
-
-  root to: 'dashboard#show'
 end
-- 
cgit v1.2.1


From 452ba19cdd8e61fa568e2f6462c14b8f31c4c07b Mon Sep 17 00:00:00 2001
From: Douwe Maan <douwe@gitlab.com>
Date: Sat, 21 Feb 2015 10:58:11 +0100
Subject: Change check to only swap static middleware when it's enabled.

---
 config/initializers/static_files.rb | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

(limited to 'config')

diff --git a/config/initializers/static_files.rb b/config/initializers/static_files.rb
index bc4fe14bc1a..d9042c652bb 100644
--- a/config/initializers/static_files.rb
+++ b/config/initializers/static_files.rb
@@ -1,6 +1,6 @@
-begin
-  app = Rails.application
+app = Rails.application
 
+if app.config.serve_static_assets
   # The `ActionDispatch::Static` middleware intercepts requests for static files 
   # by checking if they exist in the `/public` directory. 
   # We're replacing it with our `Gitlab::Middleware::Static` that does the same,
@@ -12,7 +12,4 @@ begin
     app.paths["public"].first, 
     app.config.static_cache_control
   )
-rescue
-  # If ActionDispatch::Static wasn't loaded onto the stack (like in production), 
-  # an exception is raised.
 end
-- 
cgit v1.2.1


From c6860a5828fe569f6a81e2c96bb7e4a32f572a29 Mon Sep 17 00:00:00 2001
From: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Date: Mon, 23 Feb 2015 15:18:45 -0800
Subject: Fix style issue for rubocop

---
 config/routes.rb | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

(limited to 'config')

diff --git a/config/routes.rb b/config/routes.rb
index a3f047e36a4..f0979eac906 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -81,13 +81,13 @@ Gitlab::Application.routes.draw do
 
   scope path: :uploads do
     # Note attachments and User/Group/Project avatars
-    get ":model/:mounted_as/:id/:filename", 
-        to:           "uploads#show", 
+    get ":model/:mounted_as/:id/:filename",
+        to:           "uploads#show",
         constraints:  { model: /note|user|group|project/, mounted_as: /avatar|attachment/, filename: /.+/ }
 
     # Project markdown uploads
-    get ":id/:secret/:filename", 
-        to:           "projects/uploads#show", 
+    get ":id/:secret/:filename",
+        to:           "projects/uploads#show",
         constraints:  { id: /[a-zA-Z.0-9_\-]+\/[a-zA-Z.0-9_\-]+/, filename: /.+/ }
   end
 
@@ -148,7 +148,8 @@ Gitlab::Application.routes.draw do
     resources :namespaces, path: '/projects', constraints: { id: /[a-zA-Z.0-9_\-]+/ }, only: [] do
       root to: 'projects#index', as: :projects
 
-      resources(:projects, path: '/',
+      resources(:projects,
+                path: '/',
                 constraints: { id: /[a-zA-Z.0-9_\-]+/ },
                 only: [:index, :show]) do
         root to: 'projects#show'
@@ -268,12 +269,15 @@ Gitlab::Application.routes.draw do
         post '/preview/*id', to: 'blob#preview', constraints: { id: /.+/ }, as: 'preview_blob'
 
         scope do
-          get('/blob/*id/diff', to: 'blob#diff',
+          get('/blob/*id/diff',
+              to: 'blob#diff',
               constraints: { id: /.+/, format: false },
               as: :blob_diff)
-          get('/blob/*id', to: 'blob#show',
+          get('/blob/*id',
+              to: 'blob#show',
               constraints: { id: /.+/, format: false }, as: :blob)
-          delete('/blob/*id', to: 'blob#destroy',
+          delete('/blob/*id',
+                 to: 'blob#destroy',
                  constraints: { id: /.+/, format: false })
         end
 
-- 
cgit v1.2.1


From b821a1bd41166c295ea1625ccf57b9bb48f61649 Mon Sep 17 00:00:00 2001
From: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Date: Mon, 23 Feb 2015 16:29:32 -0800
Subject: Fix markdown image uploader after rails update

---
 config/routes.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'config')

diff --git a/config/routes.rb b/config/routes.rb
index f0979eac906..c0dbf738c1f 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -86,9 +86,9 @@ Gitlab::Application.routes.draw do
         constraints:  { model: /note|user|group|project/, mounted_as: /avatar|attachment/, filename: /.+/ }
 
     # Project markdown uploads
-    get ":id/:secret/:filename",
-        to:           "projects/uploads#show",
-        constraints:  { id: /[a-zA-Z.0-9_\-]+\/[a-zA-Z.0-9_\-]+/, filename: /.+/ }
+    get ":namespace_id/:id/:secret/:filename",
+      to:           "projects/uploads#show",
+      constraints:  { namespace_id: /[a-zA-Z.0-9_\-]+/, id: /[a-zA-Z.0-9_\-]+/, filename: /.+/ }
   end
 
   #
-- 
cgit v1.2.1


From b0dfe434c60da7d04ddf23f7a3e85af97d377568 Mon Sep 17 00:00:00 2001
From: Marin Jankovski <maxlazio@gmail.com>
Date: Mon, 23 Feb 2015 12:38:30 -0800
Subject: Using gitlab url to build links for gitlab issue tracker and add a
 spec. Fix rubocop warnings in path.

---
 config/routes.rb | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

(limited to 'config')

diff --git a/config/routes.rb b/config/routes.rb
index c0dbf738c1f..ecd439aecea 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -269,16 +269,23 @@ Gitlab::Application.routes.draw do
         post '/preview/*id', to: 'blob#preview', constraints: { id: /.+/ }, as: 'preview_blob'
 
         scope do
-          get('/blob/*id/diff',
-              to: 'blob#diff',
-              constraints: { id: /.+/, format: false },
-              as: :blob_diff)
-          get('/blob/*id',
-              to: 'blob#show',
-              constraints: { id: /.+/, format: false }, as: :blob)
-          delete('/blob/*id',
-                 to: 'blob#destroy',
-                 constraints: { id: /.+/, format: false })
+          get(
+            '/blob/*id/diff',
+            to: 'blob#diff',
+            constraints: { id: /.+/, format: false },
+            as: :blob_diff
+          )
+          get(
+            '/blob/*id',
+            to: 'blob#show',
+            constraints: { id: /.+/, format: false },
+            as: :blob
+          )
+          delete(
+            '/blob/*id',
+            to: 'blob#destroy',
+            constraints: { id: /.+/, format: false }
+          )
         end
 
         scope do
-- 
cgit v1.2.1