Merge pull request #3415 from edx/derf/fix_error_on_zero_rate_limited_agents

fix error on zero rate limited agents

Merge pull request #3415 from edx/derf/fix_error_on_zero_rate_limited_agents
fix error on zero rate limited agents
ec0ef0b6 · Fred Smith · GitHub · 28f4fe5a · 4b82ddff · ec0ef0b6
Commit ec0ef0b6 authored Oct 13, 2016 by Fred Smith Committed by GitHub Oct 13, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

playbooks/roles/nginx/templates/edx/app/nginx/sites-available/lms.j2
+3 -0

No files found.
--- a/playbooks/roles/nginx/templates/edx/app/nginx/sites-available/lms.j2
+++ b/playbooks/roles/nginx/templates/edx/app/nginx/sites-available/lms.j2
@@ -216,7 +216,10 @@ error_page {{ k }} {{ v }};
    {%- if EDXAPP_ENABLE_RATE_LIMITING -%}
    # Set Limit
    limit_req zone=cookies burst={{ EDXAPP_COURSES_REQUEST_BURST_RATE }};
+
+    {%- if EDXAPP_RATE_LIMITED_USER_AGENTS|length > 0 %}
    limit_req zone=agents burst={{ EDXAPP_COURSES_USER_AGENT_BURST_RATE }};
+    {%- endif %}
    error_page  503 = /server/rate-limit.html;
    {%- endif -%}