Skip to content

Commit 8d5628e

Browse files
committed
Added CSV separator validation
1 parent 2535269 commit 8d5628e

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

app/controllers/usage_controller.rb

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,9 +151,14 @@ def parse_filtered
151151
params[:filtered].present? && params[:filtered] == 'true'
152152
end
153153

154-
# set the csv separator or default to comma
154+
# This sets the csv separator
155+
# Ensures separator is either a comma or a safe separator
155156
def sep_param
156-
params['sep'] || ','
157+
safe_csv_separators = Rails.configuration.x.application.csv_separators
158+
sep = params['sep'].to_s
159+
return sep if safe_csv_separators.include?(sep)
160+
161+
','
157162
end
158163

159164
def min_max_dates(args:)

0 commit comments

Comments
 (0)