Fixed SQL injection

a114c988 · Shinya Maeda · d15c120f · a114c988
Commit a114c988 authored Mar 01, 2017 by Shinya Maeda
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

app/finders/pipelines_finder.rb app/finders/pipelines_finder.rb +2 -2

No files found.
--- a/app/finders/pipelines_finder.rb
+++ b/app/finders/pipelines_finder.rb
@@ -103,9 +103,9 @@ class PipelinesFinder
    if params[:order_by].present? && params[:sort].present? && 
        items.column_names.include?(params[:order_by]) && 
        (params[:sort].casecmp('ASC') || params[:sort].casecmp('DESC'))
-      items.order("#{params[:order_by]} #{params[:sort]}")
+      items.reorder(params[:order_by] => params[:sort])
    else
-      items.order(id: :desc)
+      items.reorder(id: :desc)
    end
  end
 end