[geeklog-devtalk] geeklog-devel digest, Vol 1 #312 - 1 msg

[geeklog-devel-request at lists.geeklog.net]

Send geeklog-devel mailing list submissions to
	geeklog-devel at lists.geeklog.net

To subscribe or unsubscribe via the World Wide Web, visit
	http://lists.geeklog.net/listinfo/geeklog-devel
or, via email, send a message with subject or body 'help' to
	geeklog-devel-request at lists.geeklog.net

You can reach the person managing the list at
	geeklog-devel-admin at lists.geeklog.net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of geeklog-devel digest..."


Today's Topics:

   1. Comment Spam & lib-c*.php (Michael Jervis)

--__--__--

Message: 1
From: "Michael Jervis" <mike at skinmaster.co.uk>
To: <geeklog-devel at lists.geeklog.net>
Date: Wed, 14 Apr 2004 08:07:10 +0100
Subject: [geeklog-devel] Comment Spam & lib-c*.php
Reply-To: geeklog-devel at lists.geeklog.net

Hi,

I've been having a lot of comment spam recently and I noticed there is no IP
tracking on the comments, making it hard to locate and lock out repeated
anonymous comment spam. I've modified my geeklog instance, didn't take much,
to log the comment ip address, and display it next to the delete link if you
have delete permissions.

I think this code should be a feature. Want mine?

I was thinking, as I trolled round the code changing things. IMHO
lib-custom.php and lib-common.php are the wrong way round. Sort of.

I think that your geeklog include path and custom files should be in the
same file. lib-custom.php which should be in the public_html. lib-common.php
should be in system. This way there are two touch points, config.php and
lib-custom.php where users put their stuff. They shouldn't need to edit a
'core' file, lib-common.php themselves.

The content of lib-custom.php would never need to change, it's their system
specific settings. No more editing of lib-common.php when upgrading etc. It
just make things more logical, and better for upgrading IMHO.

Just my $0.02.

Mike




--__--__--

_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://lists.geeklog.net/listinfo/geeklog-devel


End of geeklog-devel Digest