Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/adblockplus/templates/web/redirector.conf.erb

Issue 29481679: #1975 - Introduce class adblockplus::web::redirector (Closed)
Patch Set: Created July 6, 2017, 4:42 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « modules/adblockplus/manifests/web/redirector.pp ('k') | modules/private-stub/hiera/hosts.yaml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/adblockplus/templates/web/redirector.conf.erb
diff --git a/modules/adblockplus/templates/web/redirector.conf.erb b/modules/adblockplus/templates/web/redirector.conf.erb
new file mode 100644
index 0000000000000000000000000000000000000000..245876d17450b60407f1c22c8b341456af895d3d
--- /dev/null
+++ b/modules/adblockplus/templates/web/redirector.conf.erb
@@ -0,0 +1,11 @@
+# Puppet: <%= @title %>
+
+# http://hub.eyeo.com/issues/1975
+location /
+{
+<%- @targets.sort.each do |key, target| -%>
+ rewrite ^/<%= key %>(/.*)?$ "<%= target %>" break;
Fred 2017/07/06 17:17:27 Didn't we want to append the original query string
mathias 2017/07/06 17:23:54 We do. That is the default behavior of the Nginx r
+<%- end -%>
+ rewrite ^/.*$ "<%= @default %>" last;
+}
+
« no previous file with comments | « modules/adblockplus/manifests/web/redirector.pp ('k') | modules/private-stub/hiera/hosts.yaml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld