Nice - Suggestion

Posted: Sat May 29, 2004 11:19 pm

Great script so far, looks really good - and I LOVE the admin functionality and flexibility that this offers.

One suggestion, just a request more than anything - could you put the release version in the Sentinel Block, so all that see it know what version its running?

-sting

Hangin' Around Joined: May 30, 2004 Posts: 46

Very nice job on this script. Has all the functionality of all the other scripts put together without the overhead.

I run a large computer modding site and I have created 3 'G_O_D' admins, each of us having certain responsibilities. The alert emails are sent to the Site Admin. I would like to have the ability to enter the emailto address as was in the hackattempt script.

Posted: Sun May 30, 2004 8:13 am

Line 360 of the includes/sentinel.php May help

Code:

@mail($adminmail, $subject, $message,"From: $adminmail\r\nX-Mailer: "._AB_ABUSEBLOCKER."\r\n");

Mabye change it to..

Code:

@mail($adminmail, $subject, $message,"From: $adminmail, otheradmin@mysite.com, anotheradmin@mysite.com\r\nX-Mailer: "._AB_ABUSEBLOCKER."\r\n");

It should work, i didnt test it tho. Wink

Site Admin/Owner Joined: Aug 27, 2002 Posts: 17088

stephen2417 wrote:

Line 360 of the includes/sentinel.php May help

Code:

@mail($adminmail, $subject, $message,"From: $adminmail\r\nX-Mailer: "._AB_ABUSEBLOCKER."\r\n");

Mabye change it to..

Code:

@mail($adminmail, $subject, $message,"From: $adminmail, otheradmin@mysite.com, anotheradmin@mysite.com\r\nX-Mailer: "._AB_ABUSEBLOCKER."\r\n");

It should work, i didnt test it tho. Wink

Give the man a cigar! Idea

Posted: Sun May 30, 2004 8:30 am

hehe well i also just realized that i read it wrong..

If you would like to change that all togehter check out line 315 Very Happy

Posted: Sun May 30, 2004 9:13 am

sting wrote:

One suggestion, just a request more than anything - could you put the release version in the Sentinel Block, so all that see it know what version its running?

Also another suggestion, I see you have this running on your site raven. Mabye we could make the IP linked to a whois database..

EDIT just happened to look at the code and do see that you allow admins to see it. Why not users?

Posted: Sun May 30, 2004 10:41 am

Me? I have it set to all visitors.

Posted: Sun May 30, 2004 10:46 am

No no.. Look at the actual block code.. You will notice that when your an admin you get to click on the ip to get whois info. Why couldnt we make it for all users to see.

Posted: Sun May 30, 2004 10:49 am

Here like this...

Code:

<?php





if (eregi("block-Sentinel.php",$_SERVER['PHP_SELF'])) {


    Header("Location: ../index.php");


    die();


}


global $prefix, $db;


$content = "";


$result = $db->sql_query("SELECT remote_addr, reason FROM $prefix"._nsnst_ips." ORDER BY date DESC LIMIT 10");


while (list($ip_addr, $ip_reason) = $db->sql_fetchrow($result)) {


    $content .= "<b><big>&middot;</big></b>&nbsp;<a href=\"http://www.dnsstuff.com/tools/whois.ch?ip=$ip_addr\" target=\"_blank\">$ip_addr</a>\n";


  if($ip_reason == 0) {


    $ip_reason = _AB_REASON0A;


  } elseif($ip_reason == 1) {


    $ip_reason = _AB_REASON1A;


  } elseif($ip_reason == 2) {


    $ip_reason = _AB_REASON2A;


  } elseif($ip_reason == 3) {


    $ip_reason = _AB_REASON3A;


  } elseif($ip_reason == 4) {


    $ip_reason = _AB_REASON4A;


  } elseif($ip_reason == 5) {


    $ip_reason = _AB_REASON5A;


  }


  $content .= "&nbsp;-&nbsp;$ip_reason<br />\n";


}


  $content .= "<hr><center><a href=\"http://www.nukescripts.net\"><img src=\"images/sentinel/Sentinel_Small.png\" height=\"31\" width=\"88\" border=\"0\"></a></center>\n";





?>

I just took the copyright out for now so u get the general idea.

Posted: Sun May 30, 2004 10:56 am

Oh. Well, tradition has always reserved that for an admin, but I don't have any immediate issue with it. We'll let Bob, Chat, and GanjaUK chime in here since we are all collaborating on this now.

registered · Posted: Sun May 30, 2004 11:38 am

I coded it for admins only due to the traditional style of coding such things for admin. I have no problem with coding it for all to see Smile

Means less code in the block Wink

Posted: Wed Jun 02, 2004 6:59 am

Regarding adding additional emails to the alerts, I have confirmed one of Raven's suggestions as a working fix:

http://www.ravenphpscripts.com/postt1709.html