##// END OF EJS Templates
Merged r14069 (#19325)....
Jean-Philippe Lang -
r13709:fa0cdb7515fd
parent child
Show More
@@ -1,656 +1,659
1 # Redmine - project management software
1 # Redmine - project management software
2 # Copyright (C) 2006-2015 Jean-Philippe Lang
2 # Copyright (C) 2006-2015 Jean-Philippe Lang
3 #
3 #
4 # This program is free software; you can redistribute it and/or
4 # This program is free software; you can redistribute it and/or
5 # modify it under the terms of the GNU General Public License
5 # modify it under the terms of the GNU General Public License
6 # as published by the Free Software Foundation; either version 2
6 # as published by the Free Software Foundation; either version 2
7 # of the License, or (at your option) any later version.
7 # of the License, or (at your option) any later version.
8 #
8 #
9 # This program is distributed in the hope that it will be useful,
9 # This program is distributed in the hope that it will be useful,
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 # GNU General Public License for more details.
12 # GNU General Public License for more details.
13 #
13 #
14 # You should have received a copy of the GNU General Public License
14 # You should have received a copy of the GNU General Public License
15 # along with this program; if not, write to the Free Software
15 # along with this program; if not, write to the Free Software
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17
17
18 require 'uri'
18 require 'uri'
19 require 'cgi'
19 require 'cgi'
20
20
21 class Unauthorized < Exception; end
21 class Unauthorized < Exception; end
22
22
23 class ApplicationController < ActionController::Base
23 class ApplicationController < ActionController::Base
24 include Redmine::I18n
24 include Redmine::I18n
25 include Redmine::Pagination
25 include Redmine::Pagination
26 include RoutesHelper
26 include RoutesHelper
27 helper :routes
27 helper :routes
28
28
29 class_attribute :accept_api_auth_actions
29 class_attribute :accept_api_auth_actions
30 class_attribute :accept_rss_auth_actions
30 class_attribute :accept_rss_auth_actions
31 class_attribute :model_object
31 class_attribute :model_object
32
32
33 layout 'base'
33 layout 'base'
34
34
35 protect_from_forgery
35 protect_from_forgery
36
36
37 def verify_authenticity_token
37 def verify_authenticity_token
38 unless api_request?
38 unless api_request?
39 super
39 super
40 end
40 end
41 end
41 end
42
42
43 def handle_unverified_request
43 def handle_unverified_request
44 unless api_request?
44 unless api_request?
45 super
45 super
46 cookies.delete(autologin_cookie_name)
46 cookies.delete(autologin_cookie_name)
47 self.logged_user = nil
47 self.logged_user = nil
48 set_localization
48 set_localization
49 render_error :status => 422, :message => "Invalid form authenticity token."
49 render_error :status => 422, :message => "Invalid form authenticity token."
50 end
50 end
51 end
51 end
52
52
53 before_filter :session_expiration, :user_setup, :force_logout_if_password_changed, :check_if_login_required, :check_password_change, :set_localization
53 before_filter :session_expiration, :user_setup, :force_logout_if_password_changed, :check_if_login_required, :check_password_change, :set_localization
54
54
55 rescue_from ::Unauthorized, :with => :deny_access
55 rescue_from ::Unauthorized, :with => :deny_access
56 rescue_from ::ActionView::MissingTemplate, :with => :missing_template
56 rescue_from ::ActionView::MissingTemplate, :with => :missing_template
57
57
58 include Redmine::Search::Controller
58 include Redmine::Search::Controller
59 include Redmine::MenuManager::MenuController
59 include Redmine::MenuManager::MenuController
60 helper Redmine::MenuManager::MenuHelper
60 helper Redmine::MenuManager::MenuHelper
61
61
62 def session_expiration
62 def session_expiration
63 if session[:user_id]
63 if session[:user_id]
64 if session_expired? && !try_to_autologin
64 if session_expired? && !try_to_autologin
65 set_localization(User.active.find_by_id(session[:user_id]))
65 set_localization(User.active.find_by_id(session[:user_id]))
66 reset_session
66 reset_session
67 flash[:error] = l(:error_session_expired)
67 flash[:error] = l(:error_session_expired)
68 redirect_to signin_url
68 redirect_to signin_url
69 else
69 else
70 session[:atime] = Time.now.utc.to_i
70 session[:atime] = Time.now.utc.to_i
71 end
71 end
72 end
72 end
73 end
73 end
74
74
75 def session_expired?
75 def session_expired?
76 if Setting.session_lifetime?
76 if Setting.session_lifetime?
77 unless session[:ctime] && (Time.now.utc.to_i - session[:ctime].to_i <= Setting.session_lifetime.to_i * 60)
77 unless session[:ctime] && (Time.now.utc.to_i - session[:ctime].to_i <= Setting.session_lifetime.to_i * 60)
78 return true
78 return true
79 end
79 end
80 end
80 end
81 if Setting.session_timeout?
81 if Setting.session_timeout?
82 unless session[:atime] && (Time.now.utc.to_i - session[:atime].to_i <= Setting.session_timeout.to_i * 60)
82 unless session[:atime] && (Time.now.utc.to_i - session[:atime].to_i <= Setting.session_timeout.to_i * 60)
83 return true
83 return true
84 end
84 end
85 end
85 end
86 false
86 false
87 end
87 end
88
88
89 def start_user_session(user)
89 def start_user_session(user)
90 session[:user_id] = user.id
90 session[:user_id] = user.id
91 session[:ctime] = Time.now.utc.to_i
91 session[:ctime] = Time.now.utc.to_i
92 session[:atime] = Time.now.utc.to_i
92 session[:atime] = Time.now.utc.to_i
93 if user.must_change_password?
93 if user.must_change_password?
94 session[:pwd] = '1'
94 session[:pwd] = '1'
95 end
95 end
96 end
96 end
97
97
98 def user_setup
98 def user_setup
99 # Check the settings cache for each request
99 # Check the settings cache for each request
100 Setting.check_cache
100 Setting.check_cache
101 # Find the current user
101 # Find the current user
102 User.current = find_current_user
102 User.current = find_current_user
103 logger.info(" Current user: " + (User.current.logged? ? "#{User.current.login} (id=#{User.current.id})" : "anonymous")) if logger
103 logger.info(" Current user: " + (User.current.logged? ? "#{User.current.login} (id=#{User.current.id})" : "anonymous")) if logger
104 end
104 end
105
105
106 # Returns the current user or nil if no user is logged in
106 # Returns the current user or nil if no user is logged in
107 # and starts a session if needed
107 # and starts a session if needed
108 def find_current_user
108 def find_current_user
109 user = nil
109 user = nil
110 unless api_request?
110 unless api_request?
111 if session[:user_id]
111 if session[:user_id]
112 # existing session
112 # existing session
113 user = (User.active.find(session[:user_id]) rescue nil)
113 user = (User.active.find(session[:user_id]) rescue nil)
114 elsif autologin_user = try_to_autologin
114 elsif autologin_user = try_to_autologin
115 user = autologin_user
115 user = autologin_user
116 elsif params[:format] == 'atom' && params[:key] && request.get? && accept_rss_auth?
116 elsif params[:format] == 'atom' && params[:key] && request.get? && accept_rss_auth?
117 # RSS key authentication does not start a session
117 # RSS key authentication does not start a session
118 user = User.find_by_rss_key(params[:key])
118 user = User.find_by_rss_key(params[:key])
119 end
119 end
120 end
120 end
121 if user.nil? && Setting.rest_api_enabled? && accept_api_auth?
121 if user.nil? && Setting.rest_api_enabled? && accept_api_auth?
122 if (key = api_key_from_request)
122 if (key = api_key_from_request)
123 # Use API key
123 # Use API key
124 user = User.find_by_api_key(key)
124 user = User.find_by_api_key(key)
125 elsif request.authorization.to_s =~ /\ABasic /i
125 elsif request.authorization.to_s =~ /\ABasic /i
126 # HTTP Basic, either username/password or API key/random
126 # HTTP Basic, either username/password or API key/random
127 authenticate_with_http_basic do |username, password|
127 authenticate_with_http_basic do |username, password|
128 user = User.try_to_login(username, password) || User.find_by_api_key(username)
128 user = User.try_to_login(username, password) || User.find_by_api_key(username)
129 end
129 end
130 if user && user.must_change_password?
130 if user && user.must_change_password?
131 render_error :message => 'You must change your password', :status => 403
131 render_error :message => 'You must change your password', :status => 403
132 return
132 return
133 end
133 end
134 end
134 end
135 # Switch user if requested by an admin user
135 # Switch user if requested by an admin user
136 if user && user.admin? && (username = api_switch_user_from_request)
136 if user && user.admin? && (username = api_switch_user_from_request)
137 su = User.find_by_login(username)
137 su = User.find_by_login(username)
138 if su && su.active?
138 if su && su.active?
139 logger.info(" User switched by: #{user.login} (id=#{user.id})") if logger
139 logger.info(" User switched by: #{user.login} (id=#{user.id})") if logger
140 user = su
140 user = su
141 else
141 else
142 render_error :message => 'Invalid X-Redmine-Switch-User header', :status => 412
142 render_error :message => 'Invalid X-Redmine-Switch-User header', :status => 412
143 end
143 end
144 end
144 end
145 end
145 end
146 user
146 user
147 end
147 end
148
148
149 def force_logout_if_password_changed
149 def force_logout_if_password_changed
150 passwd_changed_on = User.current.passwd_changed_on || Time.at(0)
150 passwd_changed_on = User.current.passwd_changed_on || Time.at(0)
151 # Make sure we force logout only for web browser sessions, not API calls
151 # Make sure we force logout only for web browser sessions, not API calls
152 # if the password was changed after the session creation.
152 # if the password was changed after the session creation.
153 if session[:user_id] && passwd_changed_on.utc.to_i > session[:ctime].to_i
153 if session[:user_id] && passwd_changed_on.utc.to_i > session[:ctime].to_i
154 reset_session
154 reset_session
155 set_localization
155 set_localization
156 flash[:error] = l(:error_session_expired)
156 flash[:error] = l(:error_session_expired)
157 redirect_to signin_url
157 redirect_to signin_url
158 end
158 end
159 end
159 end
160
160
161 def autologin_cookie_name
161 def autologin_cookie_name
162 Redmine::Configuration['autologin_cookie_name'].presence || 'autologin'
162 Redmine::Configuration['autologin_cookie_name'].presence || 'autologin'
163 end
163 end
164
164
165 def try_to_autologin
165 def try_to_autologin
166 if cookies[autologin_cookie_name] && Setting.autologin?
166 if cookies[autologin_cookie_name] && Setting.autologin?
167 # auto-login feature starts a new session
167 # auto-login feature starts a new session
168 user = User.try_to_autologin(cookies[autologin_cookie_name])
168 user = User.try_to_autologin(cookies[autologin_cookie_name])
169 if user
169 if user
170 reset_session
170 reset_session
171 start_user_session(user)
171 start_user_session(user)
172 end
172 end
173 user
173 user
174 end
174 end
175 end
175 end
176
176
177 # Sets the logged in user
177 # Sets the logged in user
178 def logged_user=(user)
178 def logged_user=(user)
179 reset_session
179 reset_session
180 if user && user.is_a?(User)
180 if user && user.is_a?(User)
181 User.current = user
181 User.current = user
182 start_user_session(user)
182 start_user_session(user)
183 else
183 else
184 User.current = User.anonymous
184 User.current = User.anonymous
185 end
185 end
186 end
186 end
187
187
188 # Logs out current user
188 # Logs out current user
189 def logout_user
189 def logout_user
190 if User.current.logged?
190 if User.current.logged?
191 cookies.delete(autologin_cookie_name)
191 cookies.delete(autologin_cookie_name)
192 Token.delete_all(["user_id = ? AND action = ?", User.current.id, 'autologin'])
192 Token.delete_all(["user_id = ? AND action = ?", User.current.id, 'autologin'])
193 self.logged_user = nil
193 self.logged_user = nil
194 end
194 end
195 end
195 end
196
196
197 # check if login is globally required to access the application
197 # check if login is globally required to access the application
198 def check_if_login_required
198 def check_if_login_required
199 # no check needed if user is already logged in
199 # no check needed if user is already logged in
200 return true if User.current.logged?
200 return true if User.current.logged?
201 require_login if Setting.login_required?
201 require_login if Setting.login_required?
202 end
202 end
203
203
204 def check_password_change
204 def check_password_change
205 if session[:pwd]
205 if session[:pwd]
206 if User.current.must_change_password?
206 if User.current.must_change_password?
207 redirect_to my_password_path
207 redirect_to my_password_path
208 else
208 else
209 session.delete(:pwd)
209 session.delete(:pwd)
210 end
210 end
211 end
211 end
212 end
212 end
213
213
214 def set_localization(user=User.current)
214 def set_localization(user=User.current)
215 lang = nil
215 lang = nil
216 if user && user.logged?
216 if user && user.logged?
217 lang = find_language(user.language)
217 lang = find_language(user.language)
218 end
218 end
219 if lang.nil? && !Setting.force_default_language_for_anonymous? && request.env['HTTP_ACCEPT_LANGUAGE']
219 if lang.nil? && !Setting.force_default_language_for_anonymous? && request.env['HTTP_ACCEPT_LANGUAGE']
220 accept_lang = parse_qvalues(request.env['HTTP_ACCEPT_LANGUAGE']).first
220 accept_lang = parse_qvalues(request.env['HTTP_ACCEPT_LANGUAGE']).first
221 if !accept_lang.blank?
221 if !accept_lang.blank?
222 accept_lang = accept_lang.downcase
222 accept_lang = accept_lang.downcase
223 lang = find_language(accept_lang) || find_language(accept_lang.split('-').first)
223 lang = find_language(accept_lang) || find_language(accept_lang.split('-').first)
224 end
224 end
225 end
225 end
226 lang ||= Setting.default_language
226 lang ||= Setting.default_language
227 set_language_if_valid(lang)
227 set_language_if_valid(lang)
228 end
228 end
229
229
230 def require_login
230 def require_login
231 if !User.current.logged?
231 if !User.current.logged?
232 # Extract only the basic url parameters on non-GET requests
232 # Extract only the basic url parameters on non-GET requests
233 if request.get?
233 if request.get?
234 url = url_for(params)
234 url = url_for(params)
235 else
235 else
236 url = url_for(:controller => params[:controller], :action => params[:action], :id => params[:id], :project_id => params[:project_id])
236 url = url_for(:controller => params[:controller], :action => params[:action], :id => params[:id], :project_id => params[:project_id])
237 end
237 end
238 respond_to do |format|
238 respond_to do |format|
239 format.html {
239 format.html {
240 if request.xhr?
240 if request.xhr?
241 head :unauthorized
241 head :unauthorized
242 else
242 else
243 redirect_to :controller => "account", :action => "login", :back_url => url
243 redirect_to :controller => "account", :action => "login", :back_url => url
244 end
244 end
245 }
245 }
246 format.atom { redirect_to :controller => "account", :action => "login", :back_url => url }
246 format.any(:atom, :pdf, :csv) {
247 redirect_to :controller => "account", :action => "login", :back_url => url
248 }
247 format.xml { head :unauthorized, 'WWW-Authenticate' => 'Basic realm="Redmine API"' }
249 format.xml { head :unauthorized, 'WWW-Authenticate' => 'Basic realm="Redmine API"' }
248 format.js { head :unauthorized, 'WWW-Authenticate' => 'Basic realm="Redmine API"' }
250 format.js { head :unauthorized, 'WWW-Authenticate' => 'Basic realm="Redmine API"' }
249 format.json { head :unauthorized, 'WWW-Authenticate' => 'Basic realm="Redmine API"' }
251 format.json { head :unauthorized, 'WWW-Authenticate' => 'Basic realm="Redmine API"' }
252 format.any { head :unauthorized }
250 end
253 end
251 return false
254 return false
252 end
255 end
253 true
256 true
254 end
257 end
255
258
256 def require_admin
259 def require_admin
257 return unless require_login
260 return unless require_login
258 if !User.current.admin?
261 if !User.current.admin?
259 render_403
262 render_403
260 return false
263 return false
261 end
264 end
262 true
265 true
263 end
266 end
264
267
265 def deny_access
268 def deny_access
266 User.current.logged? ? render_403 : require_login
269 User.current.logged? ? render_403 : require_login
267 end
270 end
268
271
269 # Authorize the user for the requested action
272 # Authorize the user for the requested action
270 def authorize(ctrl = params[:controller], action = params[:action], global = false)
273 def authorize(ctrl = params[:controller], action = params[:action], global = false)
271 allowed = User.current.allowed_to?({:controller => ctrl, :action => action}, @project || @projects, :global => global)
274 allowed = User.current.allowed_to?({:controller => ctrl, :action => action}, @project || @projects, :global => global)
272 if allowed
275 if allowed
273 true
276 true
274 else
277 else
275 if @project && @project.archived?
278 if @project && @project.archived?
276 render_403 :message => :notice_not_authorized_archived_project
279 render_403 :message => :notice_not_authorized_archived_project
277 else
280 else
278 deny_access
281 deny_access
279 end
282 end
280 end
283 end
281 end
284 end
282
285
283 # Authorize the user for the requested action outside a project
286 # Authorize the user for the requested action outside a project
284 def authorize_global(ctrl = params[:controller], action = params[:action], global = true)
287 def authorize_global(ctrl = params[:controller], action = params[:action], global = true)
285 authorize(ctrl, action, global)
288 authorize(ctrl, action, global)
286 end
289 end
287
290
288 # Find project of id params[:id]
291 # Find project of id params[:id]
289 def find_project
292 def find_project
290 @project = Project.find(params[:id])
293 @project = Project.find(params[:id])
291 rescue ActiveRecord::RecordNotFound
294 rescue ActiveRecord::RecordNotFound
292 render_404
295 render_404
293 end
296 end
294
297
295 # Find project of id params[:project_id]
298 # Find project of id params[:project_id]
296 def find_project_by_project_id
299 def find_project_by_project_id
297 @project = Project.find(params[:project_id])
300 @project = Project.find(params[:project_id])
298 rescue ActiveRecord::RecordNotFound
301 rescue ActiveRecord::RecordNotFound
299 render_404
302 render_404
300 end
303 end
301
304
302 # Find a project based on params[:project_id]
305 # Find a project based on params[:project_id]
303 # TODO: some subclasses override this, see about merging their logic
306 # TODO: some subclasses override this, see about merging their logic
304 def find_optional_project
307 def find_optional_project
305 @project = Project.find(params[:project_id]) unless params[:project_id].blank?
308 @project = Project.find(params[:project_id]) unless params[:project_id].blank?
306 allowed = User.current.allowed_to?({:controller => params[:controller], :action => params[:action]}, @project, :global => true)
309 allowed = User.current.allowed_to?({:controller => params[:controller], :action => params[:action]}, @project, :global => true)
307 allowed ? true : deny_access
310 allowed ? true : deny_access
308 rescue ActiveRecord::RecordNotFound
311 rescue ActiveRecord::RecordNotFound
309 render_404
312 render_404
310 end
313 end
311
314
312 # Finds and sets @project based on @object.project
315 # Finds and sets @project based on @object.project
313 def find_project_from_association
316 def find_project_from_association
314 render_404 unless @object.present?
317 render_404 unless @object.present?
315
318
316 @project = @object.project
319 @project = @object.project
317 end
320 end
318
321
319 def find_model_object
322 def find_model_object
320 model = self.class.model_object
323 model = self.class.model_object
321 if model
324 if model
322 @object = model.find(params[:id])
325 @object = model.find(params[:id])
323 self.instance_variable_set('@' + controller_name.singularize, @object) if @object
326 self.instance_variable_set('@' + controller_name.singularize, @object) if @object
324 end
327 end
325 rescue ActiveRecord::RecordNotFound
328 rescue ActiveRecord::RecordNotFound
326 render_404
329 render_404
327 end
330 end
328
331
329 def self.model_object(model)
332 def self.model_object(model)
330 self.model_object = model
333 self.model_object = model
331 end
334 end
332
335
333 # Find the issue whose id is the :id parameter
336 # Find the issue whose id is the :id parameter
334 # Raises a Unauthorized exception if the issue is not visible
337 # Raises a Unauthorized exception if the issue is not visible
335 def find_issue
338 def find_issue
336 # Issue.visible.find(...) can not be used to redirect user to the login form
339 # Issue.visible.find(...) can not be used to redirect user to the login form
337 # if the issue actually exists but requires authentication
340 # if the issue actually exists but requires authentication
338 @issue = Issue.find(params[:id])
341 @issue = Issue.find(params[:id])
339 raise Unauthorized unless @issue.visible?
342 raise Unauthorized unless @issue.visible?
340 @project = @issue.project
343 @project = @issue.project
341 rescue ActiveRecord::RecordNotFound
344 rescue ActiveRecord::RecordNotFound
342 render_404
345 render_404
343 end
346 end
344
347
345 # Find issues with a single :id param or :ids array param
348 # Find issues with a single :id param or :ids array param
346 # Raises a Unauthorized exception if one of the issues is not visible
349 # Raises a Unauthorized exception if one of the issues is not visible
347 def find_issues
350 def find_issues
348 @issues = Issue.where(:id => (params[:id] || params[:ids])).preload(:project, :status, :tracker, :priority, :author, :assigned_to, :relations_to).to_a
351 @issues = Issue.where(:id => (params[:id] || params[:ids])).preload(:project, :status, :tracker, :priority, :author, :assigned_to, :relations_to).to_a
349 raise ActiveRecord::RecordNotFound if @issues.empty?
352 raise ActiveRecord::RecordNotFound if @issues.empty?
350 raise Unauthorized unless @issues.all?(&:visible?)
353 raise Unauthorized unless @issues.all?(&:visible?)
351 @projects = @issues.collect(&:project).compact.uniq
354 @projects = @issues.collect(&:project).compact.uniq
352 @project = @projects.first if @projects.size == 1
355 @project = @projects.first if @projects.size == 1
353 rescue ActiveRecord::RecordNotFound
356 rescue ActiveRecord::RecordNotFound
354 render_404
357 render_404
355 end
358 end
356
359
357 def find_attachments
360 def find_attachments
358 if (attachments = params[:attachments]).present?
361 if (attachments = params[:attachments]).present?
359 att = attachments.values.collect do |attachment|
362 att = attachments.values.collect do |attachment|
360 Attachment.find_by_token( attachment[:token] ) if attachment[:token].present?
363 Attachment.find_by_token( attachment[:token] ) if attachment[:token].present?
361 end
364 end
362 att.compact!
365 att.compact!
363 end
366 end
364 @attachments = att || []
367 @attachments = att || []
365 end
368 end
366
369
367 # make sure that the user is a member of the project (or admin) if project is private
370 # make sure that the user is a member of the project (or admin) if project is private
368 # used as a before_filter for actions that do not require any particular permission on the project
371 # used as a before_filter for actions that do not require any particular permission on the project
369 def check_project_privacy
372 def check_project_privacy
370 if @project && !@project.archived?
373 if @project && !@project.archived?
371 if @project.visible?
374 if @project.visible?
372 true
375 true
373 else
376 else
374 deny_access
377 deny_access
375 end
378 end
376 else
379 else
377 @project = nil
380 @project = nil
378 render_404
381 render_404
379 false
382 false
380 end
383 end
381 end
384 end
382
385
383 def back_url
386 def back_url
384 url = params[:back_url]
387 url = params[:back_url]
385 if url.nil? && referer = request.env['HTTP_REFERER']
388 if url.nil? && referer = request.env['HTTP_REFERER']
386 url = CGI.unescape(referer.to_s)
389 url = CGI.unescape(referer.to_s)
387 end
390 end
388 url
391 url
389 end
392 end
390
393
391 def redirect_back_or_default(default, options={})
394 def redirect_back_or_default(default, options={})
392 back_url = params[:back_url].to_s
395 back_url = params[:back_url].to_s
393 if back_url.present? && valid_back_url?(back_url)
396 if back_url.present? && valid_back_url?(back_url)
394 redirect_to(back_url)
397 redirect_to(back_url)
395 return
398 return
396 elsif options[:referer]
399 elsif options[:referer]
397 redirect_to_referer_or default
400 redirect_to_referer_or default
398 return
401 return
399 end
402 end
400 redirect_to default
403 redirect_to default
401 false
404 false
402 end
405 end
403
406
404 # Returns true if back_url is a valid url for redirection, otherwise false
407 # Returns true if back_url is a valid url for redirection, otherwise false
405 def valid_back_url?(back_url)
408 def valid_back_url?(back_url)
406 if CGI.unescape(back_url).include?('..')
409 if CGI.unescape(back_url).include?('..')
407 return false
410 return false
408 end
411 end
409
412
410 begin
413 begin
411 uri = URI.parse(back_url)
414 uri = URI.parse(back_url)
412 rescue URI::InvalidURIError
415 rescue URI::InvalidURIError
413 return false
416 return false
414 end
417 end
415
418
416 if uri.host.present? && uri.host != request.host
419 if uri.host.present? && uri.host != request.host
417 return false
420 return false
418 end
421 end
419
422
420 if uri.path.match(%r{/(login|account/register)})
423 if uri.path.match(%r{/(login|account/register)})
421 return false
424 return false
422 end
425 end
423
426
424 if relative_url_root.present? && !uri.path.starts_with?(relative_url_root)
427 if relative_url_root.present? && !uri.path.starts_with?(relative_url_root)
425 return false
428 return false
426 end
429 end
427
430
428 return true
431 return true
429 end
432 end
430 private :valid_back_url?
433 private :valid_back_url?
431
434
432 # Redirects to the request referer if present, redirects to args or call block otherwise.
435 # Redirects to the request referer if present, redirects to args or call block otherwise.
433 def redirect_to_referer_or(*args, &block)
436 def redirect_to_referer_or(*args, &block)
434 redirect_to :back
437 redirect_to :back
435 rescue ::ActionController::RedirectBackError
438 rescue ::ActionController::RedirectBackError
436 if args.any?
439 if args.any?
437 redirect_to *args
440 redirect_to *args
438 elsif block_given?
441 elsif block_given?
439 block.call
442 block.call
440 else
443 else
441 raise "#redirect_to_referer_or takes arguments or a block"
444 raise "#redirect_to_referer_or takes arguments or a block"
442 end
445 end
443 end
446 end
444
447
445 def render_403(options={})
448 def render_403(options={})
446 @project = nil
449 @project = nil
447 render_error({:message => :notice_not_authorized, :status => 403}.merge(options))
450 render_error({:message => :notice_not_authorized, :status => 403}.merge(options))
448 return false
451 return false
449 end
452 end
450
453
451 def render_404(options={})
454 def render_404(options={})
452 render_error({:message => :notice_file_not_found, :status => 404}.merge(options))
455 render_error({:message => :notice_file_not_found, :status => 404}.merge(options))
453 return false
456 return false
454 end
457 end
455
458
456 # Renders an error response
459 # Renders an error response
457 def render_error(arg)
460 def render_error(arg)
458 arg = {:message => arg} unless arg.is_a?(Hash)
461 arg = {:message => arg} unless arg.is_a?(Hash)
459
462
460 @message = arg[:message]
463 @message = arg[:message]
461 @message = l(@message) if @message.is_a?(Symbol)
464 @message = l(@message) if @message.is_a?(Symbol)
462 @status = arg[:status] || 500
465 @status = arg[:status] || 500
463
466
464 respond_to do |format|
467 respond_to do |format|
465 format.html {
468 format.html {
466 render :template => 'common/error', :layout => use_layout, :status => @status
469 render :template => 'common/error', :layout => use_layout, :status => @status
467 }
470 }
468 format.any { head @status }
471 format.any { head @status }
469 end
472 end
470 end
473 end
471
474
472 # Handler for ActionView::MissingTemplate exception
475 # Handler for ActionView::MissingTemplate exception
473 def missing_template
476 def missing_template
474 logger.warn "Missing template, responding with 404"
477 logger.warn "Missing template, responding with 404"
475 @project = nil
478 @project = nil
476 render_404
479 render_404
477 end
480 end
478
481
479 # Filter for actions that provide an API response
482 # Filter for actions that provide an API response
480 # but have no HTML representation for non admin users
483 # but have no HTML representation for non admin users
481 def require_admin_or_api_request
484 def require_admin_or_api_request
482 return true if api_request?
485 return true if api_request?
483 if User.current.admin?
486 if User.current.admin?
484 true
487 true
485 elsif User.current.logged?
488 elsif User.current.logged?
486 render_error(:status => 406)
489 render_error(:status => 406)
487 else
490 else
488 deny_access
491 deny_access
489 end
492 end
490 end
493 end
491
494
492 # Picks which layout to use based on the request
495 # Picks which layout to use based on the request
493 #
496 #
494 # @return [boolean, string] name of the layout to use or false for no layout
497 # @return [boolean, string] name of the layout to use or false for no layout
495 def use_layout
498 def use_layout
496 request.xhr? ? false : 'base'
499 request.xhr? ? false : 'base'
497 end
500 end
498
501
499 def render_feed(items, options={})
502 def render_feed(items, options={})
500 @items = (items || []).to_a
503 @items = (items || []).to_a
501 @items.sort! {|x,y| y.event_datetime <=> x.event_datetime }
504 @items.sort! {|x,y| y.event_datetime <=> x.event_datetime }
502 @items = @items.slice(0, Setting.feeds_limit.to_i)
505 @items = @items.slice(0, Setting.feeds_limit.to_i)
503 @title = options[:title] || Setting.app_title
506 @title = options[:title] || Setting.app_title
504 render :template => "common/feed", :formats => [:atom], :layout => false,
507 render :template => "common/feed", :formats => [:atom], :layout => false,
505 :content_type => 'application/atom+xml'
508 :content_type => 'application/atom+xml'
506 end
509 end
507
510
508 def self.accept_rss_auth(*actions)
511 def self.accept_rss_auth(*actions)
509 if actions.any?
512 if actions.any?
510 self.accept_rss_auth_actions = actions
513 self.accept_rss_auth_actions = actions
511 else
514 else
512 self.accept_rss_auth_actions || []
515 self.accept_rss_auth_actions || []
513 end
516 end
514 end
517 end
515
518
516 def accept_rss_auth?(action=action_name)
519 def accept_rss_auth?(action=action_name)
517 self.class.accept_rss_auth.include?(action.to_sym)
520 self.class.accept_rss_auth.include?(action.to_sym)
518 end
521 end
519
522
520 def self.accept_api_auth(*actions)
523 def self.accept_api_auth(*actions)
521 if actions.any?
524 if actions.any?
522 self.accept_api_auth_actions = actions
525 self.accept_api_auth_actions = actions
523 else
526 else
524 self.accept_api_auth_actions || []
527 self.accept_api_auth_actions || []
525 end
528 end
526 end
529 end
527
530
528 def accept_api_auth?(action=action_name)
531 def accept_api_auth?(action=action_name)
529 self.class.accept_api_auth.include?(action.to_sym)
532 self.class.accept_api_auth.include?(action.to_sym)
530 end
533 end
531
534
532 # Returns the number of objects that should be displayed
535 # Returns the number of objects that should be displayed
533 # on the paginated list
536 # on the paginated list
534 def per_page_option
537 def per_page_option
535 per_page = nil
538 per_page = nil
536 if params[:per_page] && Setting.per_page_options_array.include?(params[:per_page].to_s.to_i)
539 if params[:per_page] && Setting.per_page_options_array.include?(params[:per_page].to_s.to_i)
537 per_page = params[:per_page].to_s.to_i
540 per_page = params[:per_page].to_s.to_i
538 session[:per_page] = per_page
541 session[:per_page] = per_page
539 elsif session[:per_page]
542 elsif session[:per_page]
540 per_page = session[:per_page]
543 per_page = session[:per_page]
541 else
544 else
542 per_page = Setting.per_page_options_array.first || 25
545 per_page = Setting.per_page_options_array.first || 25
543 end
546 end
544 per_page
547 per_page
545 end
548 end
546
549
547 # Returns offset and limit used to retrieve objects
550 # Returns offset and limit used to retrieve objects
548 # for an API response based on offset, limit and page parameters
551 # for an API response based on offset, limit and page parameters
549 def api_offset_and_limit(options=params)
552 def api_offset_and_limit(options=params)
550 if options[:offset].present?
553 if options[:offset].present?
551 offset = options[:offset].to_i
554 offset = options[:offset].to_i
552 if offset < 0
555 if offset < 0
553 offset = 0
556 offset = 0
554 end
557 end
555 end
558 end
556 limit = options[:limit].to_i
559 limit = options[:limit].to_i
557 if limit < 1
560 if limit < 1
558 limit = 25
561 limit = 25
559 elsif limit > 100
562 elsif limit > 100
560 limit = 100
563 limit = 100
561 end
564 end
562 if offset.nil? && options[:page].present?
565 if offset.nil? && options[:page].present?
563 offset = (options[:page].to_i - 1) * limit
566 offset = (options[:page].to_i - 1) * limit
564 offset = 0 if offset < 0
567 offset = 0 if offset < 0
565 end
568 end
566 offset ||= 0
569 offset ||= 0
567
570
568 [offset, limit]
571 [offset, limit]
569 end
572 end
570
573
571 # qvalues http header parser
574 # qvalues http header parser
572 # code taken from webrick
575 # code taken from webrick
573 def parse_qvalues(value)
576 def parse_qvalues(value)
574 tmp = []
577 tmp = []
575 if value
578 if value
576 parts = value.split(/,\s*/)
579 parts = value.split(/,\s*/)
577 parts.each {|part|
580 parts.each {|part|
578 if m = %r{^([^\s,]+?)(?:;\s*q=(\d+(?:\.\d+)?))?$}.match(part)
581 if m = %r{^([^\s,]+?)(?:;\s*q=(\d+(?:\.\d+)?))?$}.match(part)
579 val = m[1]
582 val = m[1]
580 q = (m[2] or 1).to_f
583 q = (m[2] or 1).to_f
581 tmp.push([val, q])
584 tmp.push([val, q])
582 end
585 end
583 }
586 }
584 tmp = tmp.sort_by{|val, q| -q}
587 tmp = tmp.sort_by{|val, q| -q}
585 tmp.collect!{|val, q| val}
588 tmp.collect!{|val, q| val}
586 end
589 end
587 return tmp
590 return tmp
588 rescue
591 rescue
589 nil
592 nil
590 end
593 end
591
594
592 # Returns a string that can be used as filename value in Content-Disposition header
595 # Returns a string that can be used as filename value in Content-Disposition header
593 def filename_for_content_disposition(name)
596 def filename_for_content_disposition(name)
594 request.env['HTTP_USER_AGENT'] =~ %r{(MSIE|Trident)} ? ERB::Util.url_encode(name) : name
597 request.env['HTTP_USER_AGENT'] =~ %r{(MSIE|Trident)} ? ERB::Util.url_encode(name) : name
595 end
598 end
596
599
597 def api_request?
600 def api_request?
598 %w(xml json).include? params[:format]
601 %w(xml json).include? params[:format]
599 end
602 end
600
603
601 # Returns the API key present in the request
604 # Returns the API key present in the request
602 def api_key_from_request
605 def api_key_from_request
603 if params[:key].present?
606 if params[:key].present?
604 params[:key].to_s
607 params[:key].to_s
605 elsif request.headers["X-Redmine-API-Key"].present?
608 elsif request.headers["X-Redmine-API-Key"].present?
606 request.headers["X-Redmine-API-Key"].to_s
609 request.headers["X-Redmine-API-Key"].to_s
607 end
610 end
608 end
611 end
609
612
610 # Returns the API 'switch user' value if present
613 # Returns the API 'switch user' value if present
611 def api_switch_user_from_request
614 def api_switch_user_from_request
612 request.headers["X-Redmine-Switch-User"].to_s.presence
615 request.headers["X-Redmine-Switch-User"].to_s.presence
613 end
616 end
614
617
615 # Renders a warning flash if obj has unsaved attachments
618 # Renders a warning flash if obj has unsaved attachments
616 def render_attachment_warning_if_needed(obj)
619 def render_attachment_warning_if_needed(obj)
617 flash[:warning] = l(:warning_attachments_not_saved, obj.unsaved_attachments.size) if obj.unsaved_attachments.present?
620 flash[:warning] = l(:warning_attachments_not_saved, obj.unsaved_attachments.size) if obj.unsaved_attachments.present?
618 end
621 end
619
622
620 # Rescues an invalid query statement. Just in case...
623 # Rescues an invalid query statement. Just in case...
621 def query_statement_invalid(exception)
624 def query_statement_invalid(exception)
622 logger.error "Query::StatementInvalid: #{exception.message}" if logger
625 logger.error "Query::StatementInvalid: #{exception.message}" if logger
623 session.delete(:query)
626 session.delete(:query)
624 sort_clear if respond_to?(:sort_clear)
627 sort_clear if respond_to?(:sort_clear)
625 render_error "An error occurred while executing the query and has been logged. Please report this error to your Redmine administrator."
628 render_error "An error occurred while executing the query and has been logged. Please report this error to your Redmine administrator."
626 end
629 end
627
630
628 # Renders a 200 response for successfull updates or deletions via the API
631 # Renders a 200 response for successfull updates or deletions via the API
629 def render_api_ok
632 def render_api_ok
630 render_api_head :ok
633 render_api_head :ok
631 end
634 end
632
635
633 # Renders a head API response
636 # Renders a head API response
634 def render_api_head(status)
637 def render_api_head(status)
635 # #head would return a response body with one space
638 # #head would return a response body with one space
636 render :text => '', :status => status, :layout => nil
639 render :text => '', :status => status, :layout => nil
637 end
640 end
638
641
639 # Renders API response on validation failure
642 # Renders API response on validation failure
640 # for an object or an array of objects
643 # for an object or an array of objects
641 def render_validation_errors(objects)
644 def render_validation_errors(objects)
642 messages = Array.wrap(objects).map {|object| object.errors.full_messages}.flatten
645 messages = Array.wrap(objects).map {|object| object.errors.full_messages}.flatten
643 render_api_errors(messages)
646 render_api_errors(messages)
644 end
647 end
645
648
646 def render_api_errors(*messages)
649 def render_api_errors(*messages)
647 @error_messages = messages.flatten
650 @error_messages = messages.flatten
648 render :template => 'common/error_messages.api', :status => :unprocessable_entity, :layout => nil
651 render :template => 'common/error_messages.api', :status => :unprocessable_entity, :layout => nil
649 end
652 end
650
653
651 # Overrides #_include_layout? so that #render with no arguments
654 # Overrides #_include_layout? so that #render with no arguments
652 # doesn't use the layout for api requests
655 # doesn't use the layout for api requests
653 def _include_layout?(*args)
656 def _include_layout?(*args)
654 api_request? ? false : super
657 api_request? ? false : super
655 end
658 end
656 end
659 end
@@ -1,90 +1,97
1 # Redmine - project management software
1 # Redmine - project management software
2 # Copyright (C) 2006-2015 Jean-Philippe Lang
2 # Copyright (C) 2006-2015 Jean-Philippe Lang
3 #
3 #
4 # This program is free software; you can redistribute it and/or
4 # This program is free software; you can redistribute it and/or
5 # modify it under the terms of the GNU General Public License
5 # modify it under the terms of the GNU General Public License
6 # as published by the Free Software Foundation; either version 2
6 # as published by the Free Software Foundation; either version 2
7 # of the License, or (at your option) any later version.
7 # of the License, or (at your option) any later version.
8 #
8 #
9 # This program is distributed in the hope that it will be useful,
9 # This program is distributed in the hope that it will be useful,
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 # GNU General Public License for more details.
12 # GNU General Public License for more details.
13 #
13 #
14 # You should have received a copy of the GNU General Public License
14 # You should have received a copy of the GNU General Public License
15 # along with this program; if not, write to the Free Software
15 # along with this program; if not, write to the Free Software
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17
17
18 require File.expand_path('../../test_helper', __FILE__)
18 require File.expand_path('../../test_helper', __FILE__)
19
19
20 class ApplicationTest < Redmine::IntegrationTest
20 class ApplicationTest < Redmine::IntegrationTest
21 include Redmine::I18n
21 include Redmine::I18n
22
22
23 fixtures :projects, :trackers, :issue_statuses, :issues,
23 fixtures :projects, :trackers, :issue_statuses, :issues,
24 :enumerations, :users, :issue_categories,
24 :enumerations, :users, :issue_categories,
25 :projects_trackers,
25 :projects_trackers,
26 :roles,
26 :roles,
27 :member_roles,
27 :member_roles,
28 :members,
28 :members,
29 :enabled_modules
29 :enabled_modules
30
30
31 def test_set_localization
31 def test_set_localization
32 Setting.default_language = 'en'
32 Setting.default_language = 'en'
33
33
34 # a french user
34 # a french user
35 get '/projects', { }, 'HTTP_ACCEPT_LANGUAGE' => 'fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3'
35 get '/projects', { }, 'HTTP_ACCEPT_LANGUAGE' => 'fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3'
36 assert_response :success
36 assert_response :success
37 assert_select 'h2', :text => 'Projets'
37 assert_select 'h2', :text => 'Projets'
38 assert_equal :fr, current_language
38 assert_equal :fr, current_language
39 assert_select "html[lang=?]", "fr"
39 assert_select "html[lang=?]", "fr"
40
40
41 # then an italien user
41 # then an italien user
42 get '/projects', { }, 'HTTP_ACCEPT_LANGUAGE' => 'it;q=0.8,en-us;q=0.5,en;q=0.3'
42 get '/projects', { }, 'HTTP_ACCEPT_LANGUAGE' => 'it;q=0.8,en-us;q=0.5,en;q=0.3'
43 assert_response :success
43 assert_response :success
44 assert_select 'h2', :text => 'Progetti'
44 assert_select 'h2', :text => 'Progetti'
45 assert_equal :it, current_language
45 assert_equal :it, current_language
46 assert_select "html[lang=?]", "it"
46 assert_select "html[lang=?]", "it"
47
47
48 # not a supported language: default language should be used
48 # not a supported language: default language should be used
49 get '/projects', { }, 'HTTP_ACCEPT_LANGUAGE' => 'zz'
49 get '/projects', { }, 'HTTP_ACCEPT_LANGUAGE' => 'zz'
50 assert_response :success
50 assert_response :success
51 assert_select 'h2', :text => 'Projects'
51 assert_select 'h2', :text => 'Projects'
52 assert_select "html[lang=?]", "en"
52 assert_select "html[lang=?]", "en"
53 end
53 end
54
54
55 def test_token_based_access_should_not_start_session
55 def test_token_based_access_should_not_start_session
56 # issue of a private project
56 # issue of a private project
57 get '/issues/4.atom'
57 get '/issues/4.atom'
58 assert_response 302
58 assert_response 302
59
59
60 rss_key = User.find(2).rss_key
60 rss_key = User.find(2).rss_key
61 get "/issues/4.atom?key=#{rss_key}"
61 get "/issues/4.atom?key=#{rss_key}"
62 assert_response 200
62 assert_response 200
63 assert_nil session[:user_id]
63 assert_nil session[:user_id]
64 end
64 end
65
65
66 def test_missing_template_should_respond_with_404
66 def test_missing_template_should_respond_with_404
67 get '/login.png'
67 get '/login.png'
68 assert_response 404
68 assert_response 404
69 end
69 end
70
70
71 def test_invalid_token_should_call_custom_handler
71 def test_invalid_token_should_call_custom_handler
72 ActionController::Base.allow_forgery_protection = true
72 ActionController::Base.allow_forgery_protection = true
73 post '/issues'
73 post '/issues'
74 assert_response 422
74 assert_response 422
75 assert_include "Invalid form authenticity token.", response.body
75 assert_include "Invalid form authenticity token.", response.body
76 ensure
76 ensure
77 ActionController::Base.allow_forgery_protection = false
77 ActionController::Base.allow_forgery_protection = false
78 end
78 end
79
79
80 def test_localization_should_be_set_correctly_on_invalid_token
80 def test_localization_should_be_set_correctly_on_invalid_token
81 ActionController::Base.allow_forgery_protection = true
81 ActionController::Base.allow_forgery_protection = true
82 Setting.default_language = 'en'
82 Setting.default_language = 'en'
83 post '/issues', { }, 'HTTP_ACCEPT_LANGUAGE' => 'fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3'
83 post '/issues', { }, 'HTTP_ACCEPT_LANGUAGE' => 'fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3'
84 assert_response 422
84 assert_response 422
85 assert_equal :fr, current_language
85 assert_equal :fr, current_language
86 assert_select "html[lang=?]", "fr"
86 assert_select "html[lang=?]", "fr"
87 ensure
87 ensure
88 ActionController::Base.allow_forgery_protection = false
88 ActionController::Base.allow_forgery_protection = false
89 end
89 end
90
91 def test_require_login_with_pdf_format_should_not_error
92 with_settings :login_required => '1' do
93 get '/issues/1.pdf'
94 assert_response 302
95 end
96 end
90 end
97 end
General Comments 0
You need to be logged in to leave comments. Login now