Escape user input before building regexps
This commit is contained in:
1 parent
c3cafdf1c5
commit
27e2f481df
1 file changed
+2
-2
@@ -82,12 +82,12 @@ class ClientManagementsController < CPanelController
|
||||
|
||||
def site_tickets
|
||||
if params[:category].present? && params[:keyword].present?
|
||||
regex = ::Regexp.new(".*" + params[:keyword] + ".*", "i")
|
||||
regex = ::Regexp.new(".*" + Regexp.escape(params[:keyword].to_s) + ".*", "i")
|
||||
@tickets = @site_construct.tickets.where(:category_id => params[:category], :subject => regex).order_by([:status, :desc],[:created_at, :desc]).page(params[:page]).per(10)
|
||||
elsif params[:category].present?
|
||||
@tickets = @site_construct.tickets.where(:category_id => params[:category]).order_by([:status, :desc],[:created_at, :desc]).page(params[:page]).per(10)
|
||||
elsif params[:keyword].present?
|
||||
regex = ::Regexp.new(".*" + params[:keyword] + ".*", "i")
|
||||
regex = ::Regexp.new(".*" + Regexp.escape(params[:keyword].to_s) + ".*", "i")
|
||||
@tickets = @site_construct.tickets.where(:subject => regex).order_by([:status, :desc],[:created_at, :desc]).page(params[:page]).per(10)
|
||||
else
|
||||
@tickets = @site_construct.tickets.order_by([:status, :desc],[:created_at, :desc]).page(params[:page]).per(10)
|
||||
|
||||
Reference in new issue
Block a user