| Author |
Message |
sting
Involved
Joined: Sep 23, 2003
Posts: 460
Location: Somewhere out there...
|
 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 |
|
|
   
  |
|
Captain_Computer
Hangin' Around
Joined: May 30, 2004
Posts: 46
|
| Posted:
Sun May 30, 2004 7:24 am |
|
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. |
|
|
|
|
|
stephen2417
Worker
Joined: Jan 18, 2004
Posts: 244
Location: Bristolville, OH
|
| 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, , \r\nX-Mailer: "._AB_ABUSEBLOCKER."\r\n"); |
It should work, i didnt test it tho.  |
|
|
|
|
|
Raven
Site Admin/Owner
Joined: Aug 27, 2002
Posts: 16987
Location: Kansas
|
| Posted:
Sun May 30, 2004 8:28 am |
|
| 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, , \r\nX-Mailer: "._AB_ABUSEBLOCKER."\r\n"); |
It should work, i didnt test it tho. |
Give the man a cigar!  |
|
|
|
|
|
stephen2417
Worker
Joined: Jan 18, 2004
Posts: 244
Location: Bristolville, OH
|
| 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  |
|
|
|
|
|
stephen2417
Worker
Joined: Jan 18, 2004
Posts: 244
Location: Bristolville, OH
|
| 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? |
|
|
|
|
|
Raven
Site Admin/Owner
Joined: Aug 27, 2002
Posts: 16987
Location: Kansas
|
| Posted:
Sun May 30, 2004 10:41 am |
|
Me? I have it set to all visitors. |
|
|
|
|
|
stephen2417
Worker
Joined: Jan 18, 2004
Posts: 244
Location: Bristolville, OH
|
| 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. |
|
|
|
|
|
stephen2417
Worker
Joined: Jan 18, 2004
Posts: 244
Location: Bristolville, OH
|
| 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>·</big></b> <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 .= " - $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. |
|
|
|
|
|
Raven
Site Admin/Owner
Joined: Aug 27, 2002
Posts: 16987
Location: Kansas
|
| 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. |
|
|
|
|
|
BobMarion
Former Admin in Good Standing
Joined: Oct 30, 2002
Posts: 1043
Location: RedNeck Land (known as Kentucky)
|
| 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  Means less code in the block |
|
|
|
|
SmackDaddy
Involved
Joined: Jun 02, 2004
Posts: 268
Location: Englewood, OH
|
| 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:
|
|
|
|
|
|
|
|
|
|
![[Valid RSS]](http://www.ravenphpscripts.com/images/valid-rss.png "Validate my RSS feed"){kind=small}
:: 
::
