Adding a querystring to rack-rewrite response

Question

I've got a simple 301 redirect to capture all non .com domains I have registered for my site as follows:

DOMAIN = 'www.mywebsite.com'

use Rack::Rewrite do
  r301 %r{.*}, "http://#{DOMAIN}$&", :if => Proc.new {|rack_env|
    rack_env['SERVER_NAME'] != DOMAIN && ENV["RACK_ENV"] == 'production'
  }
end

I'd like to add a querystring to the response to add the original domain in the format

?utm_source=#{rack_env['SERVER_NAME']}

But can't quite work out how not to crash the server :) Can it be done & retain any original query string?

It's unlikely that anyone will hit any subpages under the main domain, but when I drop the $& from the rewrite, and replace it with my string, it blows up with no errors in the logs...

score 2 · Accepted Answer · answered Jan 28 '13 at 16:35

2

I think the reason your original code won't work is because rack_env is not available within your second argument as it's a block argument to the third. Does that make sense?

You can however pass a Proc as the second argument of a redirect, so I think something like this should work (partially tested :)

DOMAIN = 'www.mywebsite.com'

ORIGINAL_REQUEST = Proc.new do |match, rack_env|
  "#{DOMAIN}?utm_source=#{rack_env['REQUEST_URI']}"
end

use Rack::Rewrite do
  r301 %r{.*}, ORIGINAL_REQUEST, :if => Proc.new {|rack_env|
    rack_env['SERVER_NAME'] != DOMAIN && ENV["RACK_ENV"] == 'production'
  }
end

answered Jan 28 '13 at 16:35

jordelver

8,292
2
32
40

This was just about spot on! Thanks so much. I just had to change the REQUEST_URI in the first block to SERVER_NAME and it works perfectly! – Les Jan 28 '13 at 21:22
Great :) Can you award me the bounty? Thanks ;) – jordelver Jan 29 '13 at 07:04
Hopefully it's awarded it to you properly - let me know if it hasn't. – Les Jan 31 '13 at 13:48
It has, thanks :) I don't know how this bounty stuff works :) – jordelver Jan 31 '13 at 14:33

Adding a querystring to rack-rewrite response

1 Answers1