]> git.openstreetmap.org Git - rails.git/blob - test/controllers/trace_controller_test.rb
Merge branch 'rails41'
[rails.git] / test / controllers / trace_controller_test.rb
1 require 'test_helper'
2
3 class TraceControllerTest < ActionController::TestCase
4   fixtures :users, :gpx_files
5   set_fixture_class :gpx_files => Trace
6
7   def setup
8     @gpx_trace_dir = Object.send("remove_const", "GPX_TRACE_DIR")
9     Object.const_set("GPX_TRACE_DIR", File.dirname(__FILE__) + "/../traces")
10   end
11
12   def teardown
13     Object.send("remove_const", "GPX_TRACE_DIR")
14     Object.const_set("GPX_TRACE_DIR", @gpx_trace_dir)
15   end
16
17   ##
18   # test all routes which lead to this controller
19   def test_routes
20     assert_routing(
21       { :path => "/api/0.6/gpx/create", :method => :post },
22       { :controller => "trace", :action => "api_create" }
23     )
24     assert_routing(
25       { :path => "/api/0.6/gpx/1", :method => :get },
26       { :controller => "trace", :action => "api_read", :id => "1" }
27     )
28     assert_routing(
29       { :path => "/api/0.6/gpx/1", :method => :put },
30       { :controller => "trace", :action => "api_update", :id => "1" }
31     )
32     assert_routing(
33       { :path => "/api/0.6/gpx/1", :method => :delete },
34       { :controller => "trace", :action => "api_delete", :id => "1" }
35     )
36     assert_recognizes(
37       { :controller => "trace", :action => "api_read", :id => "1" },
38       { :path => "/api/0.6/gpx/1/details", :method => :get }
39     )
40     assert_routing(
41       { :path => "/api/0.6/gpx/1/data", :method => :get },
42       { :controller => "trace", :action => "api_data", :id => "1" }
43     )
44     assert_routing(
45       { :path => "/api/0.6/gpx/1/data.xml", :method => :get },
46       { :controller => "trace", :action => "api_data", :id => "1", :format => "xml" }
47     )
48
49     assert_routing(
50       { :path => "/traces", :method => :get },
51       { :controller => "trace", :action => "list" }
52     )
53     assert_routing(
54       { :path => "/traces/page/1", :method => :get },
55       { :controller => "trace", :action => "list", :page => "1" }
56     )
57     assert_routing(
58       { :path => "/traces/tag/tagname", :method => :get },
59       { :controller => "trace", :action => "list", :tag => "tagname" }
60     )
61     assert_routing(
62       { :path => "/traces/tag/tagname/page/1", :method => :get },
63       { :controller => "trace", :action => "list", :tag => "tagname", :page => "1" }
64     )
65     assert_routing(
66       { :path => "/user/username/traces", :method => :get },
67       { :controller => "trace", :action => "list", :display_name => "username" }
68     )
69     assert_routing(
70       { :path => "/user/username/traces/page/1", :method => :get },
71       { :controller => "trace", :action => "list", :display_name => "username", :page => "1" }
72     )
73     assert_routing(
74       { :path => "/user/username/traces/tag/tagname", :method => :get },
75       { :controller => "trace", :action => "list", :display_name => "username", :tag => "tagname" }
76     )
77     assert_routing(
78       { :path => "/user/username/traces/tag/tagname/page/1", :method => :get },
79       { :controller => "trace", :action => "list", :display_name => "username", :tag => "tagname", :page => "1" }
80     )
81
82     assert_routing(
83       { :path => "/traces/mine", :method => :get },
84       { :controller => "trace", :action => "mine" }
85     )
86     assert_routing(
87       { :path => "/traces/mine/page/1", :method => :get },
88       { :controller => "trace", :action => "mine", :page => "1" }
89     )
90     assert_routing(
91       { :path => "/traces/mine/tag/tagname", :method => :get },
92       { :controller => "trace", :action => "mine", :tag => "tagname" }
93     )
94     assert_routing(
95       { :path => "/traces/mine/tag/tagname/page/1", :method => :get },
96       { :controller => "trace", :action => "mine", :tag => "tagname", :page => "1" }
97     )
98
99     assert_routing(
100       { :path => "/traces/rss", :method => :get },
101       { :controller => "trace", :action => "georss", :format => :rss }
102     )
103     assert_routing(
104       { :path => "/traces/tag/tagname/rss", :method => :get },
105       { :controller => "trace", :action => "georss", :tag => "tagname", :format => :rss }
106     )
107     assert_routing(
108       { :path => "/user/username/traces/rss", :method => :get },
109       { :controller => "trace", :action => "georss", :display_name => "username", :format => :rss }
110     )
111     assert_routing(
112       { :path => "/user/username/traces/tag/tagname/rss", :method => :get },
113       { :controller => "trace", :action => "georss", :display_name => "username", :tag => "tagname", :format => :rss }
114     )
115
116     assert_routing(
117       { :path => "/user/username/traces/1", :method => :get },
118       { :controller => "trace", :action => "view", :display_name => "username", :id => "1" }
119     )
120     assert_routing(
121       { :path => "/user/username/traces/1/picture", :method => :get },
122       { :controller => "trace", :action => "picture", :display_name => "username", :id => "1" }
123     )
124     assert_routing(
125       { :path => "/user/username/traces/1/icon", :method => :get },
126       { :controller => "trace", :action => "icon", :display_name => "username", :id => "1" }
127     )
128
129     assert_routing(
130       { :path => "/trace/create", :method => :get },
131       { :controller => "trace", :action => "create" }
132     )
133     assert_routing(
134       { :path => "/trace/create", :method => :post },
135       { :controller => "trace", :action => "create" }
136     )
137     assert_routing(
138       { :path => "/trace/1/data", :method => :get },
139       { :controller => "trace", :action => "data", :id => "1" }
140     )
141     assert_routing(
142       { :path => "/trace/1/data.xml", :method => :get },
143       { :controller => "trace", :action => "data", :id => "1", :format => "xml" }
144     )
145     assert_routing(
146       { :path => "/trace/1/edit", :method => :get },
147       { :controller => "trace", :action => "edit", :id => "1" }
148     )
149     assert_routing(
150       { :path => "/trace/1/edit", :method => :post },
151       { :controller => "trace", :action => "edit", :id => "1" }
152     )
153     assert_routing(
154       { :path => "/trace/1/edit", :method => :patch },
155       { :controller => "trace", :action => "edit", :id => "1" }
156     )
157     assert_routing(
158       { :path => "/trace/1/delete", :method => :post },
159       { :controller => "trace", :action => "delete", :id => "1" }
160     )
161   end
162
163   # Check that the list of changesets is displayed
164   def test_list
165     get :list
166     check_trace_list Trace.visible_to_all
167
168     get :list, :tag => "London"
169     check_trace_list Trace.tagged("London").visible_to_all
170   end
171
172   # Check that I can get mine
173   def test_list_mine
174     # First try to get it when not logged in
175     get :mine
176     assert_redirected_to :controller => 'user', :action => 'login', :referer => '/traces/mine'
177
178     # Now try when logged in
179     get :mine, {}, {:user => users(:public_user).id}
180     assert_redirected_to :controller => 'trace', :action => 'list', :display_name => users(:public_user).display_name
181
182     # Fetch the actual list
183     get :list, {:display_name => users(:public_user).display_name}, {:user => users(:public_user).id}
184     check_trace_list users(:public_user).traces
185   end
186
187   # Check the list of changesets for a specific user
188   def test_list_user
189     # Test a user with no traces
190     get :list, :display_name => users(:second_public_user).display_name
191     check_trace_list users(:second_public_user).traces.visible_to_all
192
193     # Test a user with some traces - should see only public ones
194     get :list, :display_name => users(:public_user).display_name
195     check_trace_list users(:public_user).traces.visible_to_all
196
197     # Should still see only public ones when authenticated as another user
198     get :list, {:display_name => users(:public_user).display_name}, {:user => users(:normal_user).id}
199     check_trace_list users(:public_user).traces.visible_to_all
200
201     # Should see all traces when authenticated as the target user
202     get :list, {:display_name => users(:public_user).display_name}, {:user => users(:public_user).id}
203     check_trace_list users(:public_user).traces
204
205     # Should only see traces with the correct tag when a tag is specified
206     get :list, {:display_name => users(:public_user).display_name, :tag => "London"}, {:user => users(:public_user).id}
207     check_trace_list users(:public_user).traces.tagged("London")
208   end
209
210   # Check that the rss loads
211   def test_rss
212     get :georss, :format => :rss
213     check_trace_feed Trace.visible_to_all
214
215     get :georss, :tag => "London", :format => :rss
216     check_trace_feed Trace.tagged("London").visible_to_all
217
218     get :georss, :display_name => users(:public_user).display_name, :format => :rss
219     check_trace_feed users(:public_user).traces.visible_to_all
220
221     get :georss, :display_name => users(:public_user).display_name, :tag => "Birmingham", :format => :rss
222     check_trace_feed users(:public_user).traces.tagged("Birmingham").visible_to_all
223   end
224
225   # Test viewing a trace
226   def test_view
227     # First with no auth, which should work since the trace is public
228     get :view, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}
229     check_trace_view gpx_files(:public_trace_file)
230
231     # Now with some other user, which should work since the trace is public
232     get :view, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:public_user).id}
233     check_trace_view gpx_files(:public_trace_file)
234
235     # And finally we should be able to do it with the owner of the trace
236     get :view, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:normal_user).id}
237     check_trace_view gpx_files(:public_trace_file)
238   end
239
240   # Check an anonymous trace can't be viewed by another user
241   def test_view_anon
242     # First with no auth
243     get :view, {:display_name => users(:public_user).display_name, :id => gpx_files(:anon_trace_file).id}
244     assert_response :redirect
245     assert_redirected_to :action => :list
246
247     # Now with some other user, which should work since the trace is anon
248     get :view, {:display_name => users(:public_user).display_name, :id => gpx_files(:anon_trace_file).id}, {:user => users(:normal_user).id}
249     assert_response :redirect
250     assert_redirected_to :action => :list
251
252     # And finally we should be able to do it with the owner of the trace
253     get :view, {:display_name => users(:public_user).display_name, :id => gpx_files(:anon_trace_file).id}, {:user => users(:public_user).id}
254     check_trace_view gpx_files(:anon_trace_file)
255   end
256
257   # Test viewing a trace that doesn't exist
258   def test_view_not_found
259     # First with no auth, which should work since the trace is public
260     get :view, {:display_name => users(:public_user).display_name, :id => 0}
261     assert_response :redirect
262     assert_redirected_to :action => :list
263
264     # Now with some other user, which should work since the trace is public
265     get :view, {:display_name => users(:public_user).display_name, :id => 0}, {:user => users(:public_user).id}
266     assert_response :redirect
267     assert_redirected_to :action => :list
268
269     # And finally we should be able to do it with the owner of the trace
270     get :view, {:display_name => users(:public_user).display_name, :id => 5}, {:user => users(:public_user).id}
271     assert_response :redirect
272     assert_redirected_to :action => :list
273   end
274
275   # Test downloading a trace
276   def test_data
277     # First with no auth, which should work since the trace is public
278     get :data, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}
279     check_trace_data gpx_files(:public_trace_file)
280
281     # Now with some other user, which should work since the trace is public
282     get :data, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:public_user).id}
283     check_trace_data gpx_files(:public_trace_file)
284
285     # And finally we should be able to do it with the owner of the trace
286     get :data, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:normal_user).id}
287     check_trace_data gpx_files(:public_trace_file)
288   end
289
290   # Test downloading a compressed trace
291   def test_data_compressed
292     # First get the data as is
293     get :data, {:display_name => users(:public_user).display_name, :id => gpx_files(:identifiable_trace_file).id}
294     check_trace_data gpx_files(:identifiable_trace_file), "application/x-gzip", "gpx.gz"
295
296     # Now ask explicitly for XML format
297     get :data, {:display_name => users(:public_user).display_name, :id => gpx_files(:identifiable_trace_file).id, :format => "xml"}
298     check_trace_data gpx_files(:identifiable_trace_file), "application/xml", "xml"
299
300     # Now ask explicitly for GPX format
301     get :data, {:display_name => users(:public_user).display_name, :id => gpx_files(:identifiable_trace_file).id, :format => "gpx"}
302     check_trace_data gpx_files(:identifiable_trace_file)
303   end
304
305   # Check an anonymous trace can't be downloaded by another user
306   def test_data_anon
307     # First with no auth
308     get :data, {:display_name => users(:public_user).display_name, :id => gpx_files(:anon_trace_file).id}
309     assert_response :not_found
310
311     # Now with some other user, which should work since the trace is anon
312     get :data, {:display_name => users(:public_user).display_name, :id => gpx_files(:anon_trace_file).id}, {:user => users(:normal_user).id}
313     assert_response :not_found
314
315     # And finally we should be able to do it with the owner of the trace
316     get :data, {:display_name => users(:public_user).display_name, :id => gpx_files(:anon_trace_file).id}, {:user => users(:public_user).id}
317     check_trace_data gpx_files(:anon_trace_file)
318   end
319
320   # Test downloading a trace that doesn't exist
321   def test_data_not_found
322     # First with no auth, which should work since the trace is public
323     get :data, {:display_name => users(:public_user).display_name, :id => 0}
324     assert_response :not_found
325
326     # Now with some other user, which should work since the trace is public
327     get :data, {:display_name => users(:public_user).display_name, :id => 0}, {:user => users(:public_user).id}
328     assert_response :not_found
329
330     # And finally we should be able to do it with the owner of the trace
331     get :data, {:display_name => users(:public_user).display_name, :id => 5}, {:user => users(:public_user).id}
332     assert_response :not_found
333   end
334
335   # Test fetching the edit page for a trace
336   def test_edit_get
337     # First with no auth
338     get :edit, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}
339     assert_response :redirect
340     assert_redirected_to :controller => :user, :action => :login, :referer => trace_edit_path(:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id)
341
342     # Now with some other user, which should fail
343     get :edit, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:public_user).id}
344     assert_response :forbidden
345
346     # Now with a trace which doesn't exist
347     get :edit, {:display_name => users(:public_user).display_name, :id => 0}, {:user => users(:public_user).id}
348     assert_response :not_found
349
350     # Now with a trace which has been deleted
351     get :edit, {:display_name => users(:public_user).display_name, :id => gpx_files(:deleted_trace_file).id}, {:user => users(:public_user).id}
352     assert_response :not_found
353
354     # Finally with a trace that we are allowed to edit
355     get :edit, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:normal_user).id}
356     assert_response :success
357   end
358
359   # Test saving edits to a trace
360   def test_edit_post
361     # New details
362     new_details = { :description => "Changed description", :tagstring => "new_tag", :visibility => "private" }
363
364     # First with no auth
365     post :edit, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id, :trace => new_details}
366     assert_response :forbidden
367
368     # Now with some other user, which should fail
369     post :edit, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id, :trace => new_details}, {:user => users(:public_user).id}
370     assert_response :forbidden
371
372     # Now with a trace which doesn't exist
373     post :edit, {:display_name => users(:public_user).display_name, :id => 0}, {:user => users(:public_user).id, :trace => new_details}
374     assert_response :not_found
375
376     # Now with a trace which has been deleted
377     post :edit, {:display_name => users(:public_user).display_name, :id => gpx_files(:deleted_trace_file).id, :trace => new_details}, {:user => users(:public_user).id}
378     assert_response :not_found
379
380     # Finally with a trace that we are allowed to edit
381     post :edit, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id, :trace => new_details}, {:user => users(:normal_user).id}
382     assert_response :redirect
383     assert_redirected_to :action => :view, :display_name => users(:normal_user).display_name
384     trace = Trace.find(gpx_files(:public_trace_file).id)
385     assert_equal new_details[:description], trace.description
386     assert_equal new_details[:tagstring], trace.tagstring
387     assert_equal new_details[:visibility], trace.visibility
388   end
389
390   # Test deleting a trace
391   def test_delete
392     # First with no auth
393     post :delete, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id,}
394     assert_response :forbidden
395
396     # Now with some other user, which should fail
397     post :delete, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:public_user).id}
398     assert_response :forbidden
399
400     # Now with a trace which doesn't exist
401     post :delete, {:display_name => users(:public_user).display_name, :id => 0}, {:user => users(:public_user).id}
402     assert_response :not_found
403
404     # Now with a trace has already been deleted
405     post :delete, {:display_name => users(:public_user).display_name, :id => gpx_files(:deleted_trace_file).id}, {:user => users(:public_user).id}
406     assert_response :not_found
407
408     # Finally with a trace that we are allowed to delete
409     post :delete, {:display_name => users(:normal_user).display_name, :id => gpx_files(:public_trace_file).id}, {:user => users(:normal_user).id}
410     assert_response :redirect
411     assert_redirected_to :action => :list, :display_name => users(:normal_user).display_name
412     trace = Trace.find(gpx_files(:public_trace_file).id)
413     assert_equal false, trace.visible
414   end
415
416   # Check getting a specific trace through the api
417   def test_api_read
418     # First with no auth
419     get :api_read, :id => gpx_files(:public_trace_file).id
420     assert_response :unauthorized
421
422     # Now with some other user, which should work since the trace is public
423     basic_authorization(users(:public_user).display_name, "test")
424     get :api_read, :id => gpx_files(:public_trace_file).id
425     assert_response :success
426
427     # And finally we should be able to do it with the owner of the trace
428     basic_authorization(users(:normal_user).display_name, "test")
429     get :api_read, :id => gpx_files(:public_trace_file).id
430     assert_response :success
431   end
432
433   # Check an anoymous trace can't be specifically fetched by another user
434   def test_api_read_anon
435     # Furst with no auth
436     get :api_read, :id => gpx_files(:anon_trace_file).id
437     assert_response :unauthorized
438
439     # Now try with another user, which shouldn't work since the trace is anon
440     basic_authorization(users(:normal_user).display_name, "test")
441     get :api_read, :id => gpx_files(:anon_trace_file).id
442     assert_response :forbidden
443
444     # And finally we should be able to get the trace details with the trace owner
445     basic_authorization(users(:public_user).display_name, "test")
446     get :api_read, :id => gpx_files(:anon_trace_file).id
447     assert_response :success
448   end
449
450   # Check the api details for a trace that doesn't exist
451   def test_api_read_not_found
452     # Try first with no auth, as it should requure it
453     get :api_read, :id => 0
454     assert_response :unauthorized
455
456     # Login, and try again
457     basic_authorization(users(:public_user).display_name, "test")
458     get :api_read, :id => 0
459     assert_response :not_found
460
461     # Now try a trace which did exist but has been deleted
462     basic_authorization(users(:public_user).display_name, "test")
463     get :api_read, :id => 5
464     assert_response :not_found
465   end
466
467   # Check updating a trace through the api
468   def test_api_update
469     # First with no auth
470     content gpx_files(:public_trace_file).to_xml
471     put :api_update, :id => gpx_files(:public_trace_file).id
472     assert_response :unauthorized
473
474     # Now with some other user, which should fail
475     basic_authorization(users(:public_user).display_name, "test")
476     content gpx_files(:public_trace_file).to_xml
477     put :api_update, :id => gpx_files(:public_trace_file).id
478     assert_response :forbidden
479
480     # Now with a trace which doesn't exist
481     basic_authorization(users(:public_user).display_name, "test")
482     content gpx_files(:public_trace_file).to_xml
483     put :api_update, :id => 0
484     assert_response :not_found
485
486     # Now with a trace which did exist but has been deleted
487     basic_authorization(users(:public_user).display_name, "test")
488     content gpx_files(:deleted_trace_file).to_xml
489     put :api_update, :id => gpx_files(:deleted_trace_file).id
490     assert_response :not_found
491
492     # Now try an update with the wrong ID
493     basic_authorization(users(:normal_user).display_name, "test")
494     content gpx_files(:anon_trace_file).to_xml
495     put :api_update, :id => gpx_files(:public_trace_file).id
496     assert_response :bad_request, 
497        "should not be able to update a trace with a different ID from the XML"
498
499     # And finally try an update that should work
500     basic_authorization(users(:normal_user).display_name, "test")
501     t = gpx_files(:public_trace_file)
502     t.description = "Changed description"
503     t.visibility = "private"
504     content t.to_xml
505     put :api_update, :id => t.id
506     assert_response :success
507     nt = Trace.find(t.id)
508     assert_equal nt.description, t.description
509     assert_equal nt.visibility, t.visibility
510   end
511
512   # Check deleting a trace through the api
513   def test_api_delete
514     # First with no auth
515     delete :api_delete, :id => gpx_files(:public_trace_file).id
516     assert_response :unauthorized
517
518     # Now with some other user, which should fail
519     basic_authorization(users(:public_user).display_name, "test")
520     delete :api_delete, :id => gpx_files(:public_trace_file).id
521     assert_response :forbidden
522
523     # Now with a trace which doesn't exist
524     basic_authorization(users(:public_user).display_name, "test")
525     delete :api_delete, :id => 0
526     assert_response :not_found
527
528     # And finally we should be able to do it with the owner of the trace
529     basic_authorization(users(:normal_user).display_name, "test")
530     delete :api_delete, :id => gpx_files(:public_trace_file).id
531     assert_response :success
532
533     # Try it a second time, which should fail
534     basic_authorization(users(:normal_user).display_name, "test")
535     delete :api_delete, :id => gpx_files(:public_trace_file).id
536     assert_response :not_found
537   end
538
539 private
540
541   def check_trace_feed(traces)
542     assert_response :success
543     assert_template "georss"
544     assert_equal "application/rss+xml", @response.content_type
545     assert_select "rss", :count => 1 do
546       assert_select "channel", :count => 1 do
547         assert_select "title"
548         assert_select "description"
549         assert_select "link"
550         assert_select "image"
551         assert_select "item", :count => traces.visible.count do |items|
552           traces.visible.order("timestamp DESC").zip(items).each do |trace,item|
553             assert_select item, "title", trace.name
554             assert_select item, "link", "http://test.host/user/#{trace.user.display_name}/traces/#{trace.id}"
555             assert_select item, "guid", "http://test.host/user/#{trace.user.display_name}/traces/#{trace.id}"
556             assert_select item, "description"
557 #            assert_select item, "dc:creator", trace.user.display_name
558             assert_select item, "pubDate", trace.timestamp.rfc822
559           end
560         end
561       end
562     end
563   end
564
565   def check_trace_list(traces)
566     assert_response :success
567     assert_template "list"
568
569     if traces.count > 0
570       assert_select "table#trace_list tbody", :count => 1 do
571         assert_select "tr", :count => traces.visible.count do |rows|
572           traces.visible.order("timestamp DESC").zip(rows).each do |trace,row|
573             assert_select row, "span.trace_summary", Regexp.new(Regexp.escape("(#{trace.size} points)"))
574             assert_select row, "td", Regexp.new(Regexp.escape(trace.description))
575             assert_select row, "td", Regexp.new(Regexp.escape("by #{trace.user.display_name}"))
576           end
577         end
578       end
579     else
580       assert_select "h4", /Nothing here yet/
581     end
582   end
583
584   def check_trace_view(trace)
585     assert_response :success
586     assert_template "view"
587
588     assert_select "table", :count => 1 do
589       assert_select "td", /^#{Regexp.quote(trace.name)} /
590       assert_select "td", trace.user.display_name
591       assert_select "td", trace.description
592     end
593   end
594
595   def check_trace_data(trace, content_type = "application/gpx+xml", extension = "gpx")
596     assert_response :success
597     assert_equal content_type, @response.content_type
598     assert_equal "attachment; filename=\"#{trace.id}.#{extension}\"", @response.header["Content-Disposition"]
599   end
600 end