[SOLVED] function CheckAccess () HELP!

[GreenMarch]

Hi

 

I am using the following script as an include:

<?php

 

function CheckAccess()

{

  $allowedip = '100.200.300.400';

 

  $ip = $_SERVER['REMOTE_ADDR'];

  return ($ip == $allowedip);

}

 

?>

 

Calling this from my page with:

<?php

 

include("includes/checkaccess.php");

 

if (!CheckAccess())

{

    echo 'Access denied: You do not have permission to view this page!';

  exit;

}

 

echo '';

 

?>

 

This allows me access to this page because the static IP is mine.  Everyone else just gets the Access Denied message.

 

I want to extend access to other IP addresses but cannot find the correct way to express the array.  Any ideas?

 

Thanks in advance.

 

[pocobueno1388]

Give this a try:

 

<?php

function CheckAccess()
{
  $allowedip = array('100.200.300.400', '444.333.444.555'); //add as many IP's as you would like

  $ip = $_SERVER['REMOTE_ADDR'];
  
    if (in_array($ip, $allowedip)){ 
        return TRUE;
    }
}

?>

[GreenMarch]

You're more than a little good pocobueno 1388

 

That's done the trick nicely!

 

I got pretty close myself - I just didn't think about the set of curlies around { return TRUE; }

 

Many thanks indeed!

GreenMarch

[pocobueno1388]

Your welcome =]

 

Keep in mind though, IP addresses change all the time...especially with AOL users. So a person that has access one day, may not the next. IP addresses can also be reused, so someone that did NOT have access may somehow gain access because they have a recycled IP address that has access.

[GreenMarch]

Thanks for the postscript but the site is only being accessed by a corporate network with static IPs so the issues surrounding dynamic IPs don't apply.

 

It's 6:30pm here so I'm knocking off a heading for an ice cold Corona.

 

Cheers

[JayBachatero]

I just didn't think about the set of curlies around { return TRUE; }

You don't need the brackets if its just one statement.