Checking the referrer

Question

I'm using this to check if someone came from Reddit, however it doesn't work.

var ref = document.referrer;
if(ref.match("/http://(www.)?reddit.com(/)?(.*)?/gi"){
    alert('You came from Reddit');
} else {
    alert('No you didn\'t');
}

Suggestions on the regular expression are most welcome too.

score 106 · Accepted Answer · answered Jan 08 '10 at 22:56

106

Try this:

if (ref.match(/^https?:\/\/([^\/]+\.)?reddit\.com(\/|$)/i)) {
  alert("Came from reddit");
}

The regexp:

/^           # ensure start of string
 http        # match 'http'
 s?          # 's' if it exists is okay
 :\/\/       # match '://'
 ([^\/]+\.)? # match any non '/' chars followed by a '.' (if they exist)
 reddit\.com # match 'reddit.com'
 (\/|$)      # match '/' or the end of the string
/i           # match case-insenitive

answered Jan 08 '10 at 22:56

gnarf

103,494
24
125
159

21

+1 for the in-depth explanation of the regular expression. Very helpful. – Tim Mackey Jun 28 '12 at 23:33

score 14 · Answer 2 · answered Jan 08 '10 at 22:56

14

Close your if paren...

answered Jan 08 '10 at 22:56

Skilldrick

67,147
33
171
227

3

:) Everybody makes mistakes like that. Use something that'll give you errors (like Firebug), because staring at a silently failing JavaScript page is a PITA. – Skilldrick Jan 08 '10 at 23:00

TimSmith-Aardwolf · Answer 3 · 2017-02-03T09:27:23.877

7

I've been using an alternative to RegEx by looking for the domain in the referrer

if (document.referrer.indexOf('reddit.com') >= 0) { alert('They came from Reddit.com'); }

EDIT: As thekingoftruth points out that doesn't work if reddit.com is included in an URL parameter so I've extended it a little. I've also added toLowerCase() as I spotted that in the RegExp above.

if (document.referrer.indexOf('?') > 0){
    if (document.referrer.substring(0,document.referrer.indexOf('?')).toLowerCase().indexOf('reddit.com') >= 0){
    alert('They came from Reddit');
    }
} else {
    if (document.referrer.toLowerCase().indexOf('reddit.com') > 0){
            alert('They came from Reddit');
    }
}

edited Feb 03 '17 at 09:27

answered Jun 13 '14 at 17:10

TimSmith-Aardwolf

486
5
13

2

The only issue is if the referrer contains a sub-url within it as a query parameter, say. – thekingoftruth Jul 15 '15 at 15:40
1

Fair point thekingoftruth. I suppose I could do a check for a ? and look before that. I've also added a toLowerCase(). I'm just a little sad (if not surprised) that the one line solution isn't anymore :-) – TimSmith-Aardwolf Feb 03 '17 at 09:23
I hear you. Javascript really makes it hard to have compact one-liners. :( – thekingoftruth Feb 08 '17 at 06:11

score 2 · Answer 4 · answered Jan 08 '10 at 22:56

2

Try this:

ref.match(new RegExp("^http://(www\\.)?reddit\\.com/", "i"))

Or:

ref.match(/^http:\/\/(www\.)?reddit\.com\//i)

answered Jan 08 '10 at 22:56

Gumbo

620,600
104
758
828

score 0 · Answer 5 · answered Apr 30 '20 at 10:58

0

I would use this, wouldn't it be a lesser and simply way?

  var referral= document.refferer;
   If(referral.includes("www.reddit.com"){
  alert("you came from reddit");
   }
     else{
    alert("you didn't come from reddit");
       {

answered Apr 30 '20 at 10:58

GeniusGeek

325
5
12

score -4 · Answer 6 · answered Mar 15 '14 at 18:18

-4

Use var ref = document.referer; // ONE R instead of TWO

answered Mar 15 '14 at 18:18

This matches exactly what I was looking for; the referer in the header. – cjbarth Aug 06 '15 at 11:54

Checking the referrer

6 Answers6

Linked

Related