[Hawk552]

There are major errors in this that prevent it from being approved. Here are some of them:

PHP Code:

formatex (configfile, 255, "%s/spam_blocker.ini", configfile) 


This is unacceptable and I'm not sure why this plugin actually works. This should crash the server and if it doesn't then you should consider yourself very lucky.

If you would like any information regarding possible adjustments you could make or things you could do to make this better, please feel free to post here or PM me.

Approval pending on changes.

[alexinno]

Hi TeoNoman this is a good idea , i didn't see any good working antispam plugin.
Maybe after fixing the problems above , you could add more options like

detect also IP's (numeric)
add white list (addresses that can be said on chat)
detect spam like *.bad-domain.* , *.*.bad-tld , bad-name.*.* ,etc (i think it's called regex)
allow white listed addresses like above posted the bad ones (*.good-domain.*)

sorry for my bad english or if you didn't understand what i'm asking for
thx

[TeoNoman]

What the problem? You can set a list of forbidden sites instead of whitelist. I'm not sure that there is a necessity of creating a whitelist. Also I'm not sure about IP detector. Why? Let him play. When he start spamming — the plugin will kick/ban him.

[anakin_cstrike]

Originally Posted by TeoNoman Ah... Got it. You know, we are talking about steamid and chat messages. I'm sure that strlen of steamid is less than 32, also about chat message. Anyway, I have a stock of place.

He meant to say that you can only use 32 slots: it starts from 0 to x-1, where x is the max slots of the array/string, in your case 32, so 31.

[fysiks]

Originally Posted by TeoNoman Also I'm not sure about IP detector. Why? Let him play. When he start spamming — the plugin will kick/ban him.

I think he means if someone puts an IP in chat (e.g. game server ip)

[TeoNoman]

fysiks, he can put this IP in spam_blocker.ini wut the problem?
anakin_cstrike, yes, I already understood that.

[fysiks]

Originally Posted by TeoNoman fysiks, he can put this IP in spam_blocker.ini wut the problem?

I understand that, I was just clarifying. Also, he probably means "any" IP. You can't put every IP in the list . You would have to use regex for checking for "any" IP.

Also, from what I understand, reading a file is extremely slow. You probably don't want to read the file every single time something is said. I've seen plugins that read the file on plugin_init or plugin_cfg to populate an array. This would be much less time consuming (for the processor, not the coder ).

Also, in your method the file is not closed for all situations, it's closed for the situation where nothing happens.

[TeoNoman]

Thx, going to fix.
About regex — I'm not sure that everyone can handle with regular expressions.

[fysiks]

regex.inc:

PHP Code:

#if AMXX_VERSION_NUM >= 175 


i.e. Version 1.75

[TeoNoman]

No-no, I mean not all users know what is regular expressions and how to handle with it.