Question

I’m trying to save the site that a user came from when they sign up. Right now I have a before_filter in my ApplicationController:

before_filter :save_referer

  def save_referer
    unless is_logged_in?
      session['referer'] = request.env["HTTP_REFERER"] unless session['referer']
    end
  end

Then when a user is created, it checks this session variable and sets it to nil. Sometimes this does not work and I’m worried there might be some unintended things happening with using session like this. Does anyone have a better way? Or some input perhaps?

EDIT: This is the logic I am using to save the referer:

def create     
    @user = User.new(params[:user])  
    if @user.save_with(session[:referer])
    ....
end

User

def save_with(referer)
    self.referer = referer unless referer == "null"
    self.save   
end

Is there any reason why this should not work?

Answer 1

I think there’s a flaw in your approach. As long as the user is hitting pages and is not logged in, the filter code will run. So the only way session['referer'] will not be nil is if they go straight to the signup page where they (presumably) post their login info and you check the session var.

I think you probably need to only check the referer once – to do this you’ll have to modify your filter code.

def save_referer
  unless is_logged_in?
    unless session['referer']
      session['referer'] = request.env["HTTP_REFERER"] || 'none'
    end
  end
end

Now when you want to know what their referer is, it will either be a valid url or ‘none’. Note that since it’s in the session, it’s not perfect: they could go to another url and come back and the session will still be valid.