Skip to content

Commit e7c2c36

Browse files
authored
use request.values instead of request.args -- (#1031)
request.args is just for GET variables, request.values combines GET and POST variables
1 parent ccad94a commit e7c2c36

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/server/endpoints/covidcast.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@ def gen():
353353

354354
@bp.route("/csv", methods=("GET", "POST"))
355355
def handle_export():
356-
source, signal = request.args.get("signal", "jhu-csse:confirmed_incidence_num").split(":")
356+
source, signal = request.values.get("signal", "jhu-csse:confirmed_incidence_num").split(":")
357357
source_signal_pairs = [SourceSignalPair(source, [signal])]
358358
daily_signals, weekly_signals = count_signal_time_types(source_signal_pairs)
359359
source_signal_pairs, alias_mapper = create_source_signal_alias_mapper(source_signal_pairs)
@@ -365,13 +365,13 @@ def handle_export():
365365
raise ValidationFailedException("mixing weeks with day arguments")
366366
_verify_argument_time_type_matches(is_day, daily_signals, weekly_signals)
367367

368-
geo_type = request.args.get("geo_type", "county")
369-
geo_values = request.args.get("geo_values", "*")
368+
geo_type = request.values.get("geo_type", "county")
369+
geo_values = request.values.get("geo_values", "*")
370370

371371
if geo_values != "*":
372372
geo_values = geo_values.split(",")
373373

374-
as_of, is_as_of_day = (parse_day_or_week_arg("as_of").time_values[0], parse_day_or_week_arg("as_of").is_day) if "as_of" in request.args else (None, is_day)
374+
as_of, is_as_of_day = (parse_day_or_week_arg("as_of").time_values[0], parse_day_or_week_arg("as_of").is_day) if "as_of" in request.values else (None, is_day)
375375
if is_day != is_as_of_day:
376376
raise ValidationFailedException("mixing weeks with day arguments")
377377

@@ -598,7 +598,7 @@ def handle_coverage():
598598
daily_signals, weekly_signals = count_signal_time_types(source_signal_pairs)
599599
source_signal_pairs, alias_mapper = create_source_signal_alias_mapper(source_signal_pairs)
600600

601-
geo_type = request.args.get("geo_type", "county")
601+
geo_type = request.values.get("geo_type", "county")
602602
if "window" in request.values:
603603
time_window = parse_day_or_week_range_arg("window")
604604
is_day = time_window.is_day

0 commit comments

Comments
 (0)